Nutch
  1. Nutch
  2. NUTCH-1308

Unnecessary truncate content configuration, and logging in parse-zip/ZipParser

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 1.4, nutchgora
    • Fix Version/s: 2.4
    • Component/s: parser
    • Labels:
      None

      Description

      Two issues here...

      1) Recently ferdy committed NUTCH-965 which skips parsing of truncated documents. Parse zip has it's own implementation for the same when it should really draw on the aforementioned implementation.
      2) If (in the offending piece of code mentioned above) truncation occurs, we get an incorrect log message the "Parser can't handle incomplete pdf files"!!! This is incorrect, shouldn't be there, and should be removed.

      
      72      if (contentLen != null && contentInBytes.length != len) {
      73 	return new ParseStatus(ParseStatus.FAILED,
      74 	ParseStatus.FAILED_TRUNCATED, "Content truncated at "
      75 	+ contentInBytes.length
      76 	+ " bytes. Parser can't handle incomplete pdf file.")
      77 	.getEmptyParseResult(content.getUrl(), getConf());
      78 	}
      

      For clarity, the issue is present in both Nutchgora branch[1] and Nutch trunk[2]

      [1] https://svn.apache.org/viewvc/nutch/branches/nutchgora/src/plugin/parse-zip/src/java/org/apache/nutch/parse/zip/ZipParser.java?diff_format=h&view=markup
      [2] https://svn.apache.org/viewvc/nutch/trunk/src/plugin/parse-zip/src/java/org/apache/nutch/parse/zip/ZipParser.java?diff_format=h&view=markup
      [2]

        Issue Links

          Activity

          Hide
          Markus Jelsma added a comment -

          20120304-push-1.6

          Show
          Markus Jelsma added a comment - 20120304-push-1.6
          Hide
          Lewis John McGibbney added a comment -

          This issue should also incorporate a main class to ZipParser which would enable adhoc parsing of files located on a local file system.

          Show
          Lewis John McGibbney added a comment - This issue should also incorporate a main class to ZipParser which would enable adhoc parsing of files located on a local file system.
          Hide
          Sebastian Nagel added a comment -

          Hi Lewis John McGibbney, is this fixed with NUTCH-1603?
          Attached a minimalist main for ZipParser (definitely useful).

          Show
          Sebastian Nagel added a comment - Hi Lewis John McGibbney , is this fixed with NUTCH-1603 ? Attached a minimalist main for ZipParser (definitely useful).
          Hide
          Lewis John McGibbney added a comment -

          Thanks Seb. I'll dig in to this ASAP.

          Show
          Lewis John McGibbney added a comment - Thanks Seb. I'll dig in to this ASAP.

            People

            • Assignee:
              Lewis John McGibbney
              Reporter:
              Lewis John McGibbney
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:

                Development