Jim Apple has uploaded a new change for review. http://gerrit.cloudera.org:8080/5094
Change subject: clang-tidy should tidy tests; fix alignas error in clang builds. ...................................................................... clang-tidy should tidy tests; fix alignas error in clang builds. run_clang_tidy.sh was mistakenly using -notests, which doesn't even compile the tests, rather than -skiptests, which compiles (but does not run) the backend tests. When I discovered this, I also found that all clang builds (including tidy and asan) had been broken by my previous alignas commit (10a4c5a2e47ec7bf1e563b6a75d1bf2afb47ebd8). This patch fixes that as well. Change-Id: Ib7066039f78d7ee039db619b96e25665b4d63503 --- M be/src/runtime/mem-pool.cc M bin/run_clang_tidy.sh 2 files changed, 2 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/94/5094/1 -- To view, visit http://gerrit.cloudera.org:8080/5094 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ib7066039f78d7ee039db619b96e25665b4d63503 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Jim Apple <[email protected]>
