You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Grant Henke (Code Review)" <ge...@cloudera.org> on 2019/02/08 14:16:37 UTC

[kudu-CR] KUDU-2411: Mark the KuduTestHarness as public 

Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12406


Change subject: KUDU-2411: Mark the KuduTestHarness as public
......................................................................

KUDU-2411: Mark the KuduTestHarness as public

This patch marks the KuduTestHarness as public and
evolving so that users external to Kudu can use the
rule for integration testing.

Additionally some unmarked classes were marked private
and unstable.

Change-Id: I739d5fb4a602817924187ce24d64b2561594ab37
---
M java/kudu-test-utils/src/main/java/org/apache/kudu/test/KuduTestHarness.java
M java/kudu-test-utils/src/main/java/org/apache/kudu/test/TempDirUtils.java
M java/kudu-test-utils/src/main/java/org/apache/kudu/test/cluster/KuduBinaryJarExtractor.java
3 files changed, 10 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/06/12406/1
-- 
To view, visit http://gerrit.cloudera.org:8080/12406
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I739d5fb4a602817924187ce24d64b2561594ab37
Gerrit-Change-Number: 12406
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>

[kudu-CR] KUDU-2411: Mark the KuduTestHarness as public

Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12406 )

Change subject: KUDU-2411: Mark the KuduTestHarness as public
......................................................................

KUDU-2411: Mark the KuduTestHarness as public

This patch marks the KuduTestHarness as public and
evolving so that users external to Kudu can use the
rule for integration testing.

Additionally some unmarked classes were marked private
and unstable.

Change-Id: I739d5fb4a602817924187ce24d64b2561594ab37
Reviewed-on: http://gerrit.cloudera.org:8080/12406
Tested-by: Kudu Jenkins
Reviewed-by: Brian McDevitt <br...@phdata.io>
Reviewed-by: Andrew Wong <aw...@cloudera.com>
---
M java/kudu-test-utils/src/main/java/org/apache/kudu/test/KuduTestHarness.java
M java/kudu-test-utils/src/main/java/org/apache/kudu/test/TempDirUtils.java
M java/kudu-test-utils/src/main/java/org/apache/kudu/test/cluster/KuduBinaryJarExtractor.java
3 files changed, 10 insertions(+), 2 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Brian McDevitt: Looks good to me, but someone else must approve
  Andrew Wong: Looks good to me, approved

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I739d5fb4a602817924187ce24d64b2561594ab37
Gerrit-Change-Number: 12406
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Brian McDevitt <br...@phdata.io>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>

[kudu-CR] KUDU-2411: Mark the KuduTestHarness as public

Posted by "Andrew Wong (Code Review)" <ge...@cloudera.org>.
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/12406 )

Change subject: KUDU-2411: Mark the KuduTestHarness as public
......................................................................


Patch Set 1: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I739d5fb4a602817924187ce24d64b2561594ab37
Gerrit-Change-Number: 12406
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Brian McDevitt <br...@phdata.io>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Comment-Date: Fri, 08 Feb 2019 17:03:55 +0000
Gerrit-HasComments: No

[kudu-CR] KUDU-2411: Mark the KuduTestHarness as public

Posted by "Brian McDevitt (Code Review)" <ge...@cloudera.org>.
Brian McDevitt has posted comments on this change. ( http://gerrit.cloudera.org:8080/12406 )

Change subject: KUDU-2411: Mark the KuduTestHarness as public
......................................................................


Patch Set 1: Code-Review+1


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I739d5fb4a602817924187ce24d64b2561594ab37
Gerrit-Change-Number: 12406
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Brian McDevitt <br...@phdata.io>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <mp...@apache.org>
Gerrit-Comment-Date: Fri, 08 Feb 2019 14:54:58 +0000
Gerrit-HasComments: No