----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2582/#review2970 -----------------------------------------------------------
Ship it! This looks good to me +1 /trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseInputStorageDriver.java <https://reviews.apache.org/r/2582/#comment6642> Can you add a little message, something like "Unable to initialize HBase Input StorageDriver"? Makes the stack trace a little easier to read - David On 2011-10-31 23:51:59, Vandana Ayyalasomayajula wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2582/ > ----------------------------------------------------------- > > (Updated 2011-10-31 23:51:59) > > > Review request for hcatalog, Francis Liu and David Capwell. > > > Summary > ------- > > The current input storage driver for HBase does not use columns of the output > schema for projection. The output columns should be used to read only those > specific columns from HBase. > > > This addresses bug hcatalog-143. > https://issues.apache.org/jira/browse/hcatalog-143 > > > Diffs > ----- > > > /trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseInputStorageDriver.java > 1182480 > > /trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseSerDeResultConverter.java > 1182480 > > /trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/ResultConverter.java > 1182480 > > /trunk/storage-drivers/hbase/src/test/org/apache/hcatalog/hbase/TestHBaseInputStorageDriver.java > 1182480 > > /trunk/storage-drivers/hbase/src/test/org/apache/hcatalog/hbase/TestHBaseSerDeResultConverter.java > 1182480 > > Diff: https://reviews.apache.org/r/2582/diff > > > Testing > ------- > > Unit test updates and test passes. > > > Thanks, > > Vandana > >
