Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19202 )
Change subject: [c++ build] suppress deprecation warnings ...................................................................... [c++ build] suppress deprecation warnings With newer version of Xcode (at least starting with version 13.2.1), building Kudu on macOS produces many warnings because of C++17-related deprecations, so it's not easy to spot warnings attributed to updates in the local workspace. This patch addresses the issue, adding pragmas to disable deprecated declarations where appropriate. In addition, this patch also silences warnings related to usage of the deprecated Kudu API in src/kudu/client/schema.cc (it's used just in the library implementation, so it's not a big deal). I also took the liberty to replace include guards with 'pragma once' and address some of clang-tidy's recommendations in a few affected files. Change-Id: I069a0680b4bc981b311374cc08bb950578db0dd0 Reviewed-on: http://gerrit.cloudera.org:8080/19202 Tested-by: Kudu Jenkins Reviewed-by: Yifan Zhang <[email protected]> Reviewed-by: Attila Bukor <[email protected]> --- M src/kudu/client/schema.cc M src/kudu/gutil/stl_util.h M src/kudu/security/gssapi.cc M src/kudu/util/mem_tracker.h M src/kudu/util/memory/arena.h 5 files changed, 44 insertions(+), 30 deletions(-) Approvals: Kudu Jenkins: Verified Yifan Zhang: Looks good to me, but someone else must approve Attila Bukor: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/19202 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I069a0680b4bc981b311374cc08bb950578db0dd0 Gerrit-Change-Number: 19202 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Yifan Zhang <[email protected]>
