You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@falcon.apache.org by Narayan Periwal <na...@inmobi.com> on 2015/09/18 09:13:07 UTC

Re: Review Request 37569: FALCON-1403: Revisit IT cleanup and teardown

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

(Updated Sept. 18, 2015, 7:13 a.m.)


Review request for Falcon.


Repository: falcon-git


Description
-------

Currently in many of the integration tests such as FalconCLIIT, FalconCLISmokeIT, etc.. there is not cleanup happening after the testcase runs; this leads to many entities getting accumulated in memory, which in turn leads to tests failing due to out of memory error


Diffs (updated)
-----

  webapp/src/test/java/org/apache/falcon/catalog/CatalogPartitionHandlerIT.java c7b7d3b 
  webapp/src/test/java/org/apache/falcon/catalog/HiveCatalogServiceIT.java 5faf5b5 
  webapp/src/test/java/org/apache/falcon/cli/FalconCLIIT.java b859256 
  webapp/src/test/java/org/apache/falcon/cli/FalconCLISmokeIT.java 7c1a42e 
  webapp/src/test/java/org/apache/falcon/late/LateDataHandlerIT.java ed85db0 
  webapp/src/test/java/org/apache/falcon/lifecycle/FileSystemFeedReplicationIT.java d7f36da 
  webapp/src/test/java/org/apache/falcon/lifecycle/TableStorageFeedEvictorIT.java 2a14ae4 
  webapp/src/test/java/org/apache/falcon/lifecycle/TableStorageFeedReplicationIT.java 13bc480 
  webapp/src/test/java/org/apache/falcon/process/PigProcessIT.java 575b870 
  webapp/src/test/java/org/apache/falcon/process/TableStorageProcessIT.java 2e882ea 
  webapp/src/test/java/org/apache/falcon/resource/EntityManagerJerseyIT.java 220e5a7 
  webapp/src/test/java/org/apache/falcon/resource/EntityManagerJerseySmokeIT.java d4a1d8a 
  webapp/src/test/java/org/apache/falcon/resource/EntityManagerPaginationJerseyIT.java fd78b4b 
  webapp/src/test/java/org/apache/falcon/resource/MetadataResourceJerseyIT.java 8594603 
  webapp/src/test/java/org/apache/falcon/resource/ProcessInstanceManagerIT.java d13caae 
  webapp/src/test/java/org/apache/falcon/resource/TestContext.java f031137 
  webapp/src/test/java/org/apache/falcon/validation/ClusterEntityValidationIT.java cc6615b 
  webapp/src/test/java/org/apache/falcon/validation/FeedEntityValidationIT.java f6d941c 

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


Testing
-------

Done.


Thanks,

Narayan Periwal