----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27414/#review59526 -----------------------------------------------------------
File Attachment: GribParser - GribParser.java <https://reviews.apache.org//r/27414/#fcomment45> Need a corresponding `xhtml.endElement("ul");`. File Attachment: GribParser - GribParser.java <https://reviews.apache.org//r/27414/#fcomment46> Need a corresponding `xhtml.endElement("ul");`. File Attachment: GribParser - GribParser.java <https://reviews.apache.org//r/27414/#fcomment47> Optional style comment: Can do a foreach loop. - Tyler Palsulich On Nov. 2, 2014, 3:17 p.m., Vineet Ghatge Hemantkumar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27414/ > ----------------------------------------------------------- > > (Updated Nov. 2, 2014, 3:17 p.m.) > > > Review request for tika, Lewis McGibbney, Chris Mattmann, and Tyler Palsulich. > > > Bugs: tika-1423 > https://issues.apache.org/jira/browse/tika-1423 > > > Repository: tika > > > Description > ------- > > GRIB Parser Patch > > > Diffs > ----- > > trunk/tika-parsers/pom.xml 1635045 > > Diff: https://reviews.apache.org/r/27414/diff/ > > > Testing > ------- > > To test the parser in place > 1. Download the patch and three file - GribParserTest.java, GribParser.java > and gdas.forecmwf > 2. Put the GribParser.java in the following folder - > tika-parsers/src/main/java/org/apache/tika/parser/grib. You will need to have > folder named grib here > 3. Put the GribParserTest.java in the following folder - > tika-parsers/src/test/java/org/apache/tika/parser/grib > 4. Put the resource file in the following location - > tika-parsers/src/test/resources/test-documents/ > 5. Apply the patch and build. > > > File Attachments > ---------------- > > ParserTestFile > > https://reviews.apache.org/media/uploaded/files/2014/10/31/840fcf4b-d67f-4ed5-8e7c-52d49c74c9d0__GribParserTest.java > GribParser > > https://reviews.apache.org/media/uploaded/files/2014/10/31/2f897768-d61e-4985-a254-4a45fc821524__GribParser.java > Resource file > > https://reviews.apache.org/media/uploaded/files/2014/10/31/a47d7101-98d7-4833-94f3-cdf31351e19e__gdas1.forecmwf.2014062612.grib2 > > > Thanks, > > Vineet Ghatge Hemantkumar > >
