Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12834 )
Change subject: trace: fully qualify namespace in macros ...................................................................... trace: fully qualify namespace in macros Impala imports kudu/util/ but most of the Impala code is not in the 'kudu' namespace. In order for Impala to use the TRACE() macros, it's thus necessary for them to fully qualify 'kudu::Trace' rather than assume the macro is being used in a context where 'kudu::' is in scope. Change-Id: I6365e08e85caa485611a8fdf2c329509911d49ab Reviewed-on: http://gerrit.cloudera.org:8080/12834 Tested-by: Kudu Jenkins Reviewed-by: Grant Henke <[email protected]> --- M src/kudu/util/trace.h 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Kudu Jenkins: Verified Grant Henke: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/12834 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I6365e08e85caa485611a8fdf2c329509911d49ab Gerrit-Change-Number: 12834 Gerrit-PatchSet: 3 Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Todd Lipcon <[email protected]>
