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

(Updated Dec. 9, 2014, 11:53 a.m.)


Review request for Falcon.


Changes
-------

incorporated review comments.

The reason why code is removed from CatalogStorage.dropPartitions(): It was 
optimising calls to hcat by aggregating partitions so that fewer calls are made 
to hcat. But HCatClient.dropPartitions() calls listPartitions and then drops 
each partition at a time. So, this optimisation is not required


Bugs: FALCON-419
    https://issues.apache.org/jira/browse/FALCON-419


Repository: falcon-git


Description
-------

changed HiveCatalogService to use HiveMetastoreClient instead of HCatClient.


Diffs (updated)
-----

  common/pom.xml 50dd2ea 
  common/src/main/java/org/apache/falcon/catalog/AbstractCatalogService.java 
348fac0 
  common/src/main/java/org/apache/falcon/catalog/CatalogPartition.java c5d4705 
  common/src/main/java/org/apache/falcon/catalog/HiveCatalogService.java 
3216f1e 
  common/src/main/java/org/apache/falcon/entity/CatalogStorage.java e68044a 
  pom.xml 7a8aacc 
  
prism/src/test/java/org/apache/falcon/security/FalconAuthenticationFilterTest.java
 4ceca29 
  rerun/src/main/java/org/apache/falcon/latedata/LateDataHandler.java d5b7db0 
  webapp/src/test/java/org/apache/falcon/catalog/HiveCatalogServiceIT.java 
fcf7f96 
  webapp/src/test/java/org/apache/falcon/late/LateDataHandlerIT.java c4e046b 
  
webapp/src/test/java/org/apache/falcon/lifecycle/TableStorageFeedEvictorIT.java 
d508a2d 
  webapp/src/test/java/org/apache/falcon/resource/TestContext.java 64f98d4 
  webapp/src/test/java/org/apache/falcon/util/HiveTestUtils.java 3b71f08 

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


Testing
-------

UTs


Thanks,

shwethags

Reply via email to