----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2799/#review3163 -----------------------------------------------------------
storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseBulkOutputFormat.java <https://reviews.apache.org/r/2799/#comment6985> Rather than Object it should be Writable since that is the interface it is expecting storage-drivers/hbase/src/test/org/apache/hcatalog/hbase/TestHBaseBulkOutputStorageDriver.java <https://reviews.apache.org/r/2799/#comment6986> Can you change this to WritableComparable and still have the test pass? Users should be able to work with <WritableComparable, HCatRecord> storage-drivers/hbase/src/test/org/apache/hcatalog/hbase/TestHBaseBulkOutputStorageDriver.java <https://reviews.apache.org/r/2799/#comment6987> Why BytesWritable, does it matter whats here? - David On 2011-11-10 19:50:20, Francis Liu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2799/ > ----------------------------------------------------------- > > (Updated 2011-11-10 19:50:20) > > > Review request for hcatalog, Sushanth Sowmyan, Vandana Ayyalasomayajula, and > David Capwell. > > > Summary > ------- > > Patch fixes errors thrown by explicit type checks made by SequenceFile output > format. I've updated both direct and bulk unit tests as well. > > > This addresses bug HCATALOG-155. > https://issues.apache.org/jira/browse/HCATALOG-155 > > > Diffs > ----- > > > storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseBulkOutputFormat.java > 5165f7f > > storage-drivers/hbase/src/test/org/apache/hcatalog/hbase/TestHBaseBulkOutputStorageDriver.java > c25e56d > > storage-drivers/hbase/src/test/org/apache/hcatalog/hbase/TestHBaseDirectOutputStorageDriver.java > d612584 > > Diff: https://reviews.apache.org/r/2799/diff > > > Testing > ------- > > updated tests should verify things are working as should. > > > Thanks, > > Francis > >
