Hello Andrew Wong, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/12912 to look at the new patch set (#3). Change subject: [master_sentry-itest] add description into AuthzFuncs ...................................................................... [master_sentry-itest] add description into AuthzFuncs Added 'description' field into AuthzFuncs structure so in case of test failure it's easier to find which one failed. For example, former message [ FAILED ] AuthzFuncCombinations/AuthzErrorHandlingTest.TestNonExistentTable/0, where GetParam() = 64-byte object <60-CF 44-00 00-00 00-00 00-00 00-00 00-00 00-00 70-84 46-00 00-00 00-00 D0-83 46-00 00-00 00-00 B0-CF 44-00 00-00 00-00 00-00 00-00 00-00 00-00 70-84 46-00 00-00 00-00 D0-83 46-00 00-00 00-00> became [ FAILED ] AuthzFuncCombinations/AuthzErrorHandlingTest.TestNonExistentTable/0, where GetParam() = DeleteTable This patch doesn't contain any functional changes. Change-Id: I08934c8e74b2d6f72dd703a304e48befffd2e6fc --- M src/kudu/integration-tests/master_sentry-itest.cc 1 file changed, 144 insertions(+), 129 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/12/12912/3 -- To view, visit http://gerrit.cloudera.org:8080/12912 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I08934c8e74b2d6f72dd703a304e48befffd2e6fc Gerrit-Change-Number: 12912 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>