> On 2011-09-27 17:24:53, Francis Liu wrote: > > /trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseInputFormat.java, > > line 1 > > <https://reviews.apache.org/r/2058/diff/1/?file=45816#file45816line1> > > > > I believe we have to include the apache license header in all files.
Where can I get the apache license header ? > On 2011-09-27 17:24:53, Francis Liu wrote: > > /trunk/storage-drivers/hbase/src/java/org/apache/hcatalog/hbase/HBaseConstants.java, > > line 1 > > <https://reviews.apache.org/r/2058/diff/1/?file=45815#file45815line1> > > > > since, I'm including your changes in my patch. please remove this file > > from yours or else the committer will have issues. Correct. I will update this patch. - Vandana ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2058/#review2098 ----------------------------------------------------------- On 2011-10-06 19:23:55, Vandana Ayyalasomayajula wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2058/ > ----------------------------------------------------------- > > (Updated 2011-10-06 19:23:55) > > > 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/src/java/org/apache/hcatalog/mapreduce/InitializeInput.java 1178480 > > /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 > >
