> On June 2, 2014, 2:10 p.m., Jihoon Son wrote: > > +1 > > This patch looks good to me.
Jihoon, thank you for your review. :) - Jung ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21909/#review44495 ----------------------------------------------------------- On May 26, 2014, 12:37 p.m., Jung JaeHwa wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21909/ > ----------------------------------------------------------- > > (Updated May 26, 2014, 12:37 p.m.) > > > Review request for Tajo. > > > Bugs: TAJO-849 > https://issues.apache.org/jira/browse/TAJO-849 > > > Repository: tajo > > > Description > ------- > > Currently, tajo supports Parquet storage. But users just can use this storage > with DerbyStore and MySQLStore. Thus, we need to add it to HCatalogStore. > > > Diffs > ----- > > tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/pom.xml a3cb99b > > tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/src/main/java/org/apache/tajo/catalog/store/HCatalogStore.java > 7924af1 > > tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/src/main/java/org/apache/tajo/catalog/store/HCatalogUtil.java > 98aa7c5 > > tajo-catalog/tajo-catalog-drivers/tajo-hcatalog/src/test/java/org/apache/tajo/catalog/store/TestHCatalogStore.java > 729184a > tajo-dist/pom.xml c826c57 > > Diff: https://reviews.apache.org/r/21909/diff/ > > > Testing > ------- > > mvn clean install -Phcatalog-0.12.0 > > > Thanks, > > Jung JaeHwa > >
