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>

Reply via email to