Copying dev@tika for folks to see ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Chris Mattmann, Ph.D. Chief Architect Instrument Software and Science Data Systems Section (398) NASA Jet Propulsion Laboratory Pasadena, CA 91109 USA Office: 168-519, Mailstop: 168-527 Email: [email protected] WWW: http://sunset.usc.edu/~mattmann/ ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Adjunct Associate Professor, Computer Science Department University of Southern California, Los Angeles, CA 90089 USA ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
-----Original Message----- From: Giuseppe Totaro <[email protected]> Date: Sunday, January 25, 2015 at 10:14 AM To: "[email protected]" <[email protected]> Cc: Chris Mattmann <[email protected]> Subject: Grib Parser > > > >Hello Vineet, >my name is Giuseppe and I am a PhD student of Professor Chris Mattmann. >Currently, I am working on Tika for digital forensics purposes. > > >I am writing you to ask some questions about your work on the Grib Parser. >I saw the code of the gribParser.java file and available at >https://github.com/hemantku/grib-parser. >I noted that you instantiates a File object starting from >RESOURCE_NAME_KEY string without using the InputStream object > passed to the parse method: > > >… >49 //Get grib2 file name from metadata > >50 > >51 File gribFile = new >File(metadata.get(Metadata.RESOURCE_NAME_KEY)); >52 > >53 try { > >54 NetcdfFile ncFile = >NetcdfDataset.openFile(gribFile.getAbsolutePath(), >… > > >This means that any test that does not define the RESOURCE_NAME_KEY >property in the caller as follows >metadata.add(Metadata.RESOURCE_NAME_KEY, filename); >will fail because the File constructor throws a NullPointerException. >Instead of adding RESOURCE_NAME_KEY, we can obtain the file from stream >using the TikaInputStream class as well as in NetCDFParser.java: > > > 51 //File gribFile = new >File(metadata.get(Metadata.RESOURCE_NAME_KEY)); > 53 TikaInputStream tis = TikaInputStream.get(stream, new >TemporaryResources()); > 54 > 55 try { > 57 NetcdfFile ncFile = >NetcdfDataset.openFile(tis.getFile().getAbsolutePath(), null); > > >Few minutes ago, I posted a comment on the Tika Issue Tracker system >(TIKA-1423). > > > >Furthermore, I would really appreciate if you could update me about your >progress in pushing your code into Tika? > > >Thank you for your great work. > > >Cheers, >Giuseppe
