You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@tika.apache.org by Tim Allison <ta...@mitre.org> on 2014/06/10 21:46:48 UTC

Review Request 22433: Add RecursiveParserWrapper aka Jukka's (and Nick's) RecursiveMetadataParser

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22433/
-----------------------------------------------------------

Review request for tika.


Repository: tika


Description
-------

Jukka and Nick have a great demo of parsing metadata recursively on the wiki. For TIKA-1302, I'd like to use something similar, and I think that others may find it useful for tika-app and tika-server.

I took the code from the wiki and made some modifications. I'm not sure if we should put this in parsers or in a new module for "examples." Given that I think this would be useful for tika-app and tika-server, I'd prefer parsers, but I'm open to any input...including "let's not."


Diffs
-----

  /trunk/tika-parsers/src/main/java/org/apache/tika/parser/RecursiveParserWrapper.java PRE-CREATION 
  /trunk/tika-parsers/src/test/java/org/apache/tika/parser/RecursiveParserWrapperTest.java PRE-CREATION 

Diff: https://reviews.apache.org/r/22433/diff/


Testing
-------

Included basic unit tests.


File Attachments
----------------

test file
  https://reviews.apache.org/media/uploaded/files/2014/06/10/8cd34908-0edf-47d8-8eda-b33ef4e47e7d__test_recursive_embedded.docx


Thanks,

Tim Allison