Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13638 )
Change subject: thrift: follow-up to 327be47820dcaba358095f118e516a5e92621a59 ...................................................................... thrift: follow-up to 327be47820dcaba358095f118e516a5e92621a59 Minor tweak and added some test coverage for proper ordering. Change-Id: I857a86c357c332a9cac731fa02aba8241d6cf826 Reviewed-on: http://gerrit.cloudera.org:8080/13638 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/sentry/thrift_operators-test.cc M src/kudu/sentry/thrift_operators.cc 2 files changed, 35 insertions(+), 17 deletions(-) Approvals: Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/13638 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I857a86c357c332a9cac731fa02aba8241d6cf826 Gerrit-Change-Number: 13638 Gerrit-PatchSet: 3 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
