falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Raghav Gautam" <rag...@apache.org>
Subject Review Request 26297: Fixing PrismFeedUpdateTest and FeedInstanceStatus test issues
Date Thu, 02 Oct 2014 23:59:54 GMT

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

Review request for Falcon.


Repository: falcon-git


Description
-------

Found multiple minor test issues:
EmbeddedPigScriptTest tests needs some more time on windows for oozie job creation
FeedInstanceStatusTest should have used coloname - (getColo call returns something else)
PrismFeedUpdateTest#dependentProcessSucceeded should have used cluster1's oozie client instead
of prism(which has no oozie client associated with it)


Diffs
-----

  falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/interfaces/IEntityManagerHelper.java
ec8726c 
  falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/OozieUtil.java
806bbd3 
  falcon-regression/merlin/src/test/java/org/apache/falcon/regression/EmbeddedPigScriptTest.java
99ffe37 
  falcon-regression/merlin/src/test/java/org/apache/falcon/regression/FeedInstanceStatusTest.java
3b3ba71 
  falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/EntityDryRunTest.java
55d69a3 
  falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/PrismFeedUpdateTest.java
a07161d 

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


Testing
-------

Modified EmbeddedPigScriptTest
FeedInstanceStatusTest


Thanks,

Raghav Gautam


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message