----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2058/#review2098 -----------------------------------------------------------
/trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseConstants.java <https://reviews.apache.org/r/2058/#comment4830> since, I'm including your changes in my patch. please remove this file from yours or else the committer will have issues. /trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseInputFormat.java <https://reviews.apache.org/r/2058/#comment4831> I believe we have to include the apache license header in all files. /trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseInputFormat.java <https://reviews.apache.org/r/2058/#comment4832> change your ide to use spaces instead of tabs for indentation, this messes up the code depending on where it is opened. /trunk/storage-drivers/hbase/src/test/org/apache/hcatalog/hbase/TestHBaseInputStorageDriver.java <https://reviews.apache.org/r/2058/#comment4833> space instead of tabs, no need to indent IMHO - Francis On 2011-09-26 22:06:25, Vandana Ayyalasomayajula wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2058/ > ----------------------------------------------------------- > > (Updated 2011-09-26 22:06:25) > > > Review request for hcatalog and Francis Liu. > > > Summary > ------- > > This patch deals with the input storage driver for hbase. > > > This addresses bug https://issues.apache.org/jira/browse/HCATALOG-75. > > https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/HCATALOG-75 > > > Diffs > ----- > > > /trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseConstants.java > PRE-CREATION > > /trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseInputFormat.java > PRE-CREATION > > /trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseInputStorageDriver.java > PRE-CREATION > > /trunk/storage-drivers/hbase/src/test/org/apache/hcatalog/hbase/TestHBaseInputStorageDriver.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/2058/diff > > > Testing > ------- > > I have added an additional junit test for the input storage driver. All the > existing tests along with the newly added one pass. > > > Thanks, > > Vandana > >
