Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12912 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/12912 Reviewed-by: Hao Hao <hao....@cloudera.com> Tested-by: Kudu Jenkins --- M src/kudu/integration-tests/master_sentry-itest.cc 1 file changed, 144 insertions(+), 129 deletions(-) Approvals: Hao Hao: Looks good to me, approved Kudu Jenkins: Verified -- 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: merged Gerrit-Change-Id: I08934c8e74b2d6f72dd703a304e48befffd2e6fc Gerrit-Change-Number: 12912 Gerrit-PatchSet: 4 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> Gerrit-Reviewer: Kudu Jenkins (120)