You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@kudu.apache.org by "Dan Burkert (Code Review)" <ge...@cloudera.org> on 2018/07/02 23:38:44 UTC

[kudu-CR] java: Expose test name in BaseKuduTest

Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/10808 )

Change subject: java: Expose test name in BaseKuduTest
......................................................................


Patch Set 5:

Maybe this isn't necessary given https://gerrit.cloudera.org/c/10838/ ?  I'd expect that table name conflicts should go away if we're spinning up a cluster per test case.


-- 
To view, visit http://gerrit.cloudera.org:8080/10808
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I63049288e1ab6b32fe629a5c009f2d7d523fd5c0
Gerrit-Change-Number: 10808
Gerrit-PatchSet: 5
Gerrit-Owner: Mike Percy <mp...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Comment-Date: Mon, 02 Jul 2018 23:38:44 +0000
Gerrit-HasComments: No