Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/11898 )
Change subject: De-flake sentry_client-test and sentry_authz_provider-test ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/11898/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/11898/1//COMMIT_MSG@14 PS1, Line 14: This commit sets this configuration with port 0 to ensure the mini Sentry : service can always find a free port to bound to. When running : sentry_client-test and sentry_authz_provider-test in parallel on the : same machine: before the change, one of the tests failed with : BindException; after the change, both tests passed without any errors. > Right, I meant as a regression test, but I don't feel strongly that it's ne I also don't feel strong neither, as the purpose of such test is only to ensure there wouldn't be any issues for running other tests. -- To view, visit http://gerrit.cloudera.org:8080/11898 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I0e30ce88b7c2ab4afe451c50e13ef739237709e0 Gerrit-Change-Number: 11898 Gerrit-PatchSet: 2 Gerrit-Owner: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 07 Nov 2018 20:18:09 +0000 Gerrit-HasComments: Yes