----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2056/#review2100 -----------------------------------------------------------
/trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseAuthorizationProvider.java <https://reviews.apache.org/r/2056/#comment4837> In general try to avoid having excess spaces because this will pop out as changes in a diff. /trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseHCatStorageHandler.java <https://reviews.apache.org/r/2056/#comment4838> close curly bracket should be after the end of the statement. - Francis On 2011-09-26 22:11:00, Vandana Ayyalasomayajula wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2056/ > ----------------------------------------------------------- > > (Updated 2011-09-26 22:11:00) > > > Review request for hcatalog and Francis Liu. > > > Summary > ------- > > This patch deals with the storage handler HBase in HCatalog. > > > This addresses bug https://issues.apache.org/jira/browse/HCATALOG-109. > > https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/HCATALOG-109 > > > Diffs > ----- > > > /trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseAuthorizationProvider.java > PRE-CREATION > > /trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseHCatStorageHandler.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/2056/diff > > > Testing > ------- > > No. > > > Thanks, > > Vandana > >
