Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19183 )
Change subject: IMPALA-11713: Switch to C++17 ...................................................................... IMPALA-11713: Switch to C++17 This switches from C++14 to C++17, which allows Impala code to interact with libraries written in C++17. This fixes a few minor compilation differences, such as the need for comparators to be const. Since C++17 includes -faligned-new, this removes the CMake code that specified it. Testing: - Ran core job Change-Id: Iadac41817fe5eaaa469a5f0e9f94056a409c14b9 Reviewed-on: http://gerrit.cloudera.org:8080/19183 Reviewed-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M be/CMakeLists.txt M be/src/exec/file-metadata-utils.cc M be/src/gutil/endian.h M be/src/runtime/krpc-data-stream-mgr.h M be/src/service/impala-server.h M be/src/udf_samples/CMakeLists.txt 6 files changed, 8 insertions(+), 18 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/19183 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iadac41817fe5eaaa469a5f0e9f94056a409c14b9 Gerrit-Change-Number: 19183 Gerrit-PatchSet: 7 Gerrit-Owner: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com>