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



/trunk/CHANGES.txt
<https://reviews.apache.org/r/4940/#comment21125>

    don't forget to add this before checkin



/trunk/build.xml
<https://reviews.apache.org/r/4940/#comment21243>

    use dash to separate the words to be consistent with the pig adapter



/trunk/webhcat/javaclient/ivy.xml
<https://reviews.apache.org/r/4940/#comment21245>

    you should have hcatalog-core as a dependency instead of manually adding 
the paths in the build file. Take a look at hcatalog pig adapter.



/trunk/webhcat/javaclient/src/main/java/org/apache/hcatalog/api/ConnectionLossException.java
<https://reviews.apache.org/r/4940/#comment21244>

    nitpick: ConnectionFailure?


- Francis Liu


On Aug. 6, 2012, 10:59 p.m., Vandana Ayyalasomayajula wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/4940/
> -----------------------------------------------------------
> 
> (Updated Aug. 6, 2012, 10:59 p.m.)
> 
> 
> Review request for hcatalog, Ashutosh Chauhan, Thejas Nair, Alan Gates, 
> Francis Liu, David Capwell, and Rohini Palaniswamy.
> 
> 
> Description
> -------
> 
> HCatalog java APIs for DDL commands.
> 
> 
> This addresses bug hcatalog-419.
>     https://issues.apache.org/jira/browse/hcatalog-419
> 
> 
> Diffs
> -----
> 
>   /trunk/build.xml 1369895 
>   /trunk/webhcat/javaclient/build.xml PRE-CREATION 
>   /trunk/webhcat/javaclient/ivy.xml PRE-CREATION 
>   
> /trunk/webhcat/javaclient/src/main/java/org/apache/hcatalog/api/ConnectionLossException.java
>  PRE-CREATION 
>   
> /trunk/webhcat/javaclient/src/main/java/org/apache/hcatalog/api/HCatAddPartitionDesc.java
>  PRE-CREATION 
>   
> /trunk/webhcat/javaclient/src/main/java/org/apache/hcatalog/api/HCatClient.java
>  PRE-CREATION 
>   
> /trunk/webhcat/javaclient/src/main/java/org/apache/hcatalog/api/HCatClientHMSImpl.java
>  PRE-CREATION 
>   
> /trunk/webhcat/javaclient/src/main/java/org/apache/hcatalog/api/HCatCreateDBDesc.java
>  PRE-CREATION 
>   
> /trunk/webhcat/javaclient/src/main/java/org/apache/hcatalog/api/HCatCreateTableDesc.java
>  PRE-CREATION 
>   
> /trunk/webhcat/javaclient/src/main/java/org/apache/hcatalog/api/HCatDatabase.java
>  PRE-CREATION 
>   
> /trunk/webhcat/javaclient/src/main/java/org/apache/hcatalog/api/HCatPartition.java
>  PRE-CREATION 
>   
> /trunk/webhcat/javaclient/src/main/java/org/apache/hcatalog/api/HCatTable.java
>  PRE-CREATION 
>   
> /trunk/webhcat/javaclient/src/test/java/org/apache/hcatalog/api/TestHCatClient.java
>  PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/4940/diff/
> 
> 
> Testing
> -------
> 
> Unit test passes. 
> 
> TODO:
> 
> - cover more test cases.
> - E2E tests.
> 
> 
> Thanks,
> 
> Vandana Ayyalasomayajula
> 
>

Reply via email to