You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Szehon Ho <sz...@cloudera.com> on 2014/03/06 03:50:01 UTC
Review Request 18829: HIVE-6487 PTest2 do not copy failed source directories
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18829/
-----------------------------------------------------------
Review request for hive and Brock Noland.
Bugs: HIVE-6487
https://issues.apache.org/jira/browse/HIVE-6487
Repository: hive-git
Description
-------
Removing the cp of the source directory from failed tests.
Diffs
-----
testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/HostExecutor.java 69bfb28
testutils/ptest2/src/main/resources/batch-exec.vm 9a2ebac
testutils/ptest2/src/test/java/org/apache/hive/ptest/execution/TestScripts.testAlternativeTestJVM.approved.txt c30e44b
testutils/ptest2/src/test/java/org/apache/hive/ptest/execution/TestScripts.testBatch.approved.txt 97befb1
Diff: https://reviews.apache.org/r/18829/diff/
Testing
-------
Tested manually on my own cluster.
Thanks,
Szehon Ho
Re: Review Request 18829: HIVE-6487 PTest2 do not copy failed source
directories
Posted by Brock Noland <br...@cloudera.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18829/#review36333
-----------------------------------------------------------
Ship it!
Ship It!
- Brock Noland
On March 6, 2014, 2:50 a.m., Szehon Ho wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/18829/
> -----------------------------------------------------------
>
> (Updated March 6, 2014, 2:50 a.m.)
>
>
> Review request for hive and Brock Noland.
>
>
> Bugs: HIVE-6487
> https://issues.apache.org/jira/browse/HIVE-6487
>
>
> Repository: hive-git
>
>
> Description
> -------
>
> Removing the cp of the source directory from failed tests.
>
>
> Diffs
> -----
>
> testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/HostExecutor.java 69bfb28
> testutils/ptest2/src/main/resources/batch-exec.vm 9a2ebac
> testutils/ptest2/src/test/java/org/apache/hive/ptest/execution/TestScripts.testAlternativeTestJVM.approved.txt c30e44b
> testutils/ptest2/src/test/java/org/apache/hive/ptest/execution/TestScripts.testBatch.approved.txt 97befb1
>
> Diff: https://reviews.apache.org/r/18829/diff/
>
>
> Testing
> -------
>
> Tested manually on my own cluster.
>
>
> Thanks,
>
> Szehon Ho
>
>