Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-3182: Reenable -Werror for kudu files ......................................................................
IMPALA-3182: Reenable -Werror for kudu files Now the macros DISALLOW_COPY_AND_ASSIGN and COMPILE_ASSERT are potentially defined by either the Kudu client or gutil. Either definition should be fine. This change adds an "ifndef" around gutil's definitions to avoid warning about redefining the macros. Kudu already has conditional ifdefs. Change-Id: I87c84b667c56dacc369bf9a2ff6550bb59b4fea9 Reviewed-on: http://gerrit.cloudera.org:8080/2660 Reviewed-by: Tim Armstrong <[email protected]> Reviewed-by: Dan Hecht <[email protected]> Tested-by: Internal Jenkins --- M be/src/exec/CMakeLists.txt M be/src/gutil/macros.h 2 files changed, 4 insertions(+), 19 deletions(-) Approvals: Internal Jenkins: Verified Dan Hecht: Looks good to me, approved Tim Armstrong: Looks good to me, but someone else must approve -- To view, visit http://gerrit.cloudera.org:8080/2660 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I87c84b667c56dacc369bf9a2ff6550bb59b4fea9 Gerrit-PatchSet: 2 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Casey Ching <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Tim Armstrong <[email protected]>
