Skip to content

Commit 1ac9c8a

Browse files
committed
Bug#29800785 CLANG-FORMAT ABSTRACT_QUERY_PLAN
The files abstract_query_plan.cc and abstract_query_plan.h has been left out of clang-formatting with justification that it's part of NDB. It might be true that ndbcluster plugin is the only user of the interface exposed by abstract_query_plan, but the files are compiled as part of MySQL Server and should thus adhere to the same rules as rest of MySQL Server. This is difference which just leads to confusion about which coding style should be used for these files. Fix by turning on on clang formatting of sql/abstract_query_plan files. Remove the exemption from .gitattributes Change-Id: Id524794f6d69303f47f097a4a5af13ecd65e924f
1 parent 1d4a987 commit 1ac9c8a

File tree

3 files changed

+474
-546
lines changed

3 files changed

+474
-546
lines changed

.gitattributes

-2
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,6 @@
3939
/strings/uca_data.h !filter
4040
/internal/meb/meb/mysqloption_list.cpp !filter
4141

42-
/sql/abstract_query_plan.h !filter
43-
/sql/abstract_query_plan.cc !filter
4442
# Some NDB source is currently exempt
4543
/storage/ndb/clusterj/** !filter
4644
/storage/ndb/include/** !filter

0 commit comments

Comments
 (0)