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 2021/05/24 16:17:49 UTC
[kudu-CR](branch-1.15.x) [java] a property to show output while running a test
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17494
Change subject: [java] a property to show output while running a test
......................................................................
[java] a property to show output while running a test
While working on non-trivial and relatively long running test scenarios,
I found it's inconvenient that the test output isn't shown while a test
scenario is running. This patch introduces a new command-line property
'showTestOutput' to direct gradle outputting stdout and stderr from the
JVM on the console, so it's possible to do the standard stream
redirection and other stream manipulation in a command line shell.
An example of running a particular test scenario with the output
redirected to the console:
./gradlew -DshowTestOutput cleanTest :kudu-client:test --test ...
By default the property isn't set, so this patch preserves the original
behavior w.r.t. the output of running Java tests.
Change-Id: I82590957004eef427a19f36a32e903012ea220d4
Reviewed-on: http://gerrit.cloudera.org:8080/17479
Tested-by: Kudu Jenkins
Reviewed-by: Bankim Bhavsar <ba...@cloudera.com>
Reviewed-by: Andrew Wong <aw...@cloudera.com>
(cherry picked from commit 7cdc9f7c735efe99b2d39d61ac4034d41473522e)
---
M java/gradle/tests.gradle
1 file changed, 4 insertions(+), 0 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/94/17494/1
--
To view, visit http://gerrit.cloudera.org:8080/17494
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.15.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: I82590957004eef427a19f36a32e903012ea220d4
Gerrit-Change-Number: 17494
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
[kudu-CR](branch-1.15.x) [java] a property to show output while running a test
Posted by "Bankim Bhavsar (Code Review)" <ge...@cloudera.org>.
Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/17494 )
Change subject: [java] a property to show output while running a test
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/17494
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.15.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I82590957004eef427a19f36a32e903012ea220d4
Gerrit-Change-Number: 17494
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Bankim Bhavsar <ba...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 24 May 2021 19:02:41 +0000
Gerrit-HasComments: No
[kudu-CR](branch-1.15.x) [java] a property to show output while running a test
Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17494 )
Change subject: [java] a property to show output while running a test
......................................................................
[java] a property to show output while running a test
While working on non-trivial and relatively long running test scenarios,
I found it's inconvenient that the test output isn't shown while a test
scenario is running. This patch introduces a new command-line property
'showTestOutput' to direct gradle outputting stdout and stderr from the
JVM on the console, so it's possible to do the standard stream
redirection and other stream manipulation in a command line shell.
An example of running a particular test scenario with the output
redirected to the console:
./gradlew -DshowTestOutput cleanTest :kudu-client:test --test ...
By default the property isn't set, so this patch preserves the original
behavior w.r.t. the output of running Java tests.
Change-Id: I82590957004eef427a19f36a32e903012ea220d4
Reviewed-on: http://gerrit.cloudera.org:8080/17479
Tested-by: Kudu Jenkins
Reviewed-by: Bankim Bhavsar <ba...@cloudera.com>
Reviewed-by: Andrew Wong <aw...@cloudera.com>
(cherry picked from commit 7cdc9f7c735efe99b2d39d61ac4034d41473522e)
Reviewed-on: http://gerrit.cloudera.org:8080/17494
---
M java/gradle/tests.gradle
1 file changed, 4 insertions(+), 0 deletions(-)
Approvals:
Kudu Jenkins: Verified
Bankim Bhavsar: Looks good to me, approved
--
To view, visit http://gerrit.cloudera.org:8080/17494
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.15.x
Gerrit-MessageType: merged
Gerrit-Change-Id: I82590957004eef427a19f36a32e903012ea220d4
Gerrit-Change-Number: 17494
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Bankim Bhavsar <ba...@cloudera.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)