----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2058/#review2341 -----------------------------------------------------------
/trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseInputFormat.java <https://reviews.apache.org/r/2058/#comment5401> since inputformats don't have to be specified by the user. let's make this package private. /trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseInputStorageDriver.java <https://reviews.apache.org/r/2058/#comment5402> you removed this. did you forget to update the diff? /trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseInputStorageDriver.java <https://reviews.apache.org/r/2058/#comment5403> add a todo to make this configurable? /trunk/storage-drivers/hbase/src/test/org/apache/hcatalog/hbase/TestHBaseInputStorageDriver.java <https://reviews.apache.org/r/2058/#comment5405> should this index be 2? and shouldn't the conditions be !startsWith? - Francis On 2011-09-27 21:03:19, Vandana Ayyalasomayajula wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2058/ > ----------------------------------------------------------- > > (Updated 2011-09-27 21:03:19) > > > 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/HBaseInputStorageDriver.java > PRE-CREATION > > /trunk/storage-drivers/hbase/src/test/org/apache/hcatalog/hbase/TestHBaseInputStorageDriver.java > PRE-CREATION > > /trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseInputFormat.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 > >
