Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13651 )
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]> (cherry picked from commit a7bdd06373a3862b3572db4690013c7c20a6166e) Reviewed-on: http://gerrit.cloudera.org:8080/13651 Reviewed-by: Andrew Wong <[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 Andrew Wong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/13651 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.10.x Gerrit-MessageType: merged Gerrit-Change-Id: I857a86c357c332a9cac731fa02aba8241d6cf826 Gerrit-Change-Number: 13651 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Will Berkeley <[email protected]>
