----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4940/#review8003 -----------------------------------------------------------
Ship it! Apart from adding TODO. It looks good. /trunk/src/java/org/apache/hcatalog/api/HCatHMSImpl.java <https://reviews.apache.org/r/4940/#comment17367> put a TODO that it should be moved out. /trunk/src/java/org/apache/hcatalog/api/HCatHMSImpl.java <https://reviews.apache.org/r/4940/#comment17368> TODO on this one as well - Francis On 2012-05-11 01:09:13, Vandana Ayyalasomayajula wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4940/ > ----------------------------------------------------------- > > (Updated 2012-05-11 01:09:13) > > > Review request for hcatalog, Francis Liu, David Capwell, and Rohini > Palaniswamy. > > > Summary > ------- > > HCatalog java APIs for DDL commands. > > > Diffs > ----- > > /trunk/src/java/org/apache/hcatalog/api/HCatAddPartitionDesc.java > PRE-CREATION > /trunk/src/java/org/apache/hcatalog/api/HCatClient.java PRE-CREATION > /trunk/src/java/org/apache/hcatalog/api/HCatClientHMSImpl.java PRE-CREATION > /trunk/src/java/org/apache/hcatalog/api/HCatCreateDBDesc.java PRE-CREATION > /trunk/src/java/org/apache/hcatalog/api/HCatCreateTableDesc.java > PRE-CREATION > /trunk/src/java/org/apache/hcatalog/api/HCatDatabase.java PRE-CREATION > /trunk/src/java/org/apache/hcatalog/api/HCatPartition.java PRE-CREATION > /trunk/src/java/org/apache/hcatalog/api/HCatTable.java PRE-CREATION > /trunk/src/java/org/apache/hcatalog/common/HCatConstants.java 1336918 > /trunk/src/java/org/apache/hcatalog/data/schema/HCatFieldSchema.java > 1336918 > /trunk/src/java/org/apache/hcatalog/data/schema/HCatSchemaUtils.java > 1336918 > /trunk/src/test/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 > >
