You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Brock Noland <br...@cloudera.com> on 2013/11/15 19:48:08 UTC

Review Request 15583: HIVE-5782 PTest2 should be able to ride out price spikes

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15583/
-----------------------------------------------------------

Review request for hive, Ashutosh Chauhan and Xuefu Zhang.


Bugs: HIVE-5782
    https://issues.apache.org/jira/browse/HIVE-5782


Repository: hive-git


Description
-------

Basically now we will wait forever while trying to get new hosts until we actually obtain them


Diffs
-----

  testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/context/CloudComputeService.java db1311d 
  testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/context/CloudExecutionContextProvider.java cb72992 
  testutils/ptest2/src/test/java/org/apache/hive/ptest/execution/context/TestCloudComputeService.java PRE-CREATION 
  testutils/ptest2/src/test/java/org/apache/hive/ptest/execution/context/TestCloudExecutionContextProvider.java 2711ff7 

Diff: https://reviews.apache.org/r/15583/diff/


Testing
-------

Unit tests pass


Thanks,

Brock Noland


Re: Review Request 15583: HIVE-5782 PTest2 should be able to ride out price spikes

Posted by Brock Noland <br...@cloudera.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15583/
-----------------------------------------------------------

(Updated Nov. 15, 2013, 6:50 p.m.)


Review request for hive, Ashutosh Chauhan and Xuefu Zhang.


Changes
-------

Remove trailing ws


Bugs: HIVE-5782
    https://issues.apache.org/jira/browse/HIVE-5782


Repository: hive-git


Description
-------

Basically now we will wait forever while trying to get new hosts until we actually obtain them


Diffs (updated)
-----

  testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/context/CloudComputeService.java db1311d 
  testutils/ptest2/src/main/java/org/apache/hive/ptest/execution/context/CloudExecutionContextProvider.java cb72992 
  testutils/ptest2/src/test/java/org/apache/hive/ptest/execution/context/TestCloudComputeService.java PRE-CREATION 
  testutils/ptest2/src/test/java/org/apache/hive/ptest/execution/context/TestCloudExecutionContextProvider.java 2711ff7 

Diff: https://reviews.apache.org/r/15583/diff/


Testing
-------

Unit tests pass


Thanks,

Brock Noland