You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@nutch.apache.org by "Heiko Dietze (JIRA)" <ji...@apache.org> on 2007/03/08 10:21:24 UTC

[jira] Created: (NUTCH-456) parse msexcel plugin speedup

parse msexcel plugin speedup
----------------------------

                 Key: NUTCH-456
                 URL: https://issues.apache.org/jira/browse/NUTCH-456
             Project: Nutch
          Issue Type: Improvement
          Components: fetcher
    Affects Versions: 0.9.0
         Environment: All
            Reporter: Heiko Dietze
            Priority: Minor
             Fix For: 0.9.0


The parse-msexcel plugin appends all table cells to one String. Currently this is done with += operation. Replace this with the StringBuffer. This will improve the speed of this parse plugin, because the number new objects and internal copy operations is reduced drastically.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[jira] Updated: (NUTCH-456) parse msexcel plugin speedup

Posted by "Heiko Dietze (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/NUTCH-456?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Heiko Dietze updated NUTCH-456:
-------------------------------

    Attachment: msexcel-parse_path.diff

This patch does the improvement by replace the += with StringBuffer operations.

> parse msexcel plugin speedup
> ----------------------------
>
>                 Key: NUTCH-456
>                 URL: https://issues.apache.org/jira/browse/NUTCH-456
>             Project: Nutch
>          Issue Type: Improvement
>          Components: fetcher
>    Affects Versions: 0.9.0
>         Environment: All
>            Reporter: Heiko Dietze
>            Priority: Minor
>             Fix For: 0.9.0
>
>         Attachments: msexcel-parse_path.diff
>
>
> The parse-msexcel plugin appends all table cells to one String. Currently this is done with += operation. Replace this with the StringBuffer. This will improve the speed of this parse plugin, because the number new objects and internal copy operations is reduced drastically.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[jira] Resolved: (NUTCH-456) parse msexcel plugin speedup

Posted by "Sami Siren (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/NUTCH-456?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sami Siren resolved NUTCH-456.
------------------------------

    Resolution: Fixed

committed with minor modifications (used StringBuilder instead of StringBuffer, changed text output so it matches the old version)

thanks Heiko

> parse msexcel plugin speedup
> ----------------------------
>
>                 Key: NUTCH-456
>                 URL: https://issues.apache.org/jira/browse/NUTCH-456
>             Project: Nutch
>          Issue Type: Improvement
>          Components: fetcher
>    Affects Versions: 0.9.0
>         Environment: All
>            Reporter: Heiko Dietze
>            Priority: Minor
>             Fix For: 1.0.0
>
>         Attachments: msexcel-parse_path.diff
>
>
> The parse-msexcel plugin appends all table cells to one String. Currently this is done with += operation. Replace this with the StringBuffer. This will improve the speed of this parse plugin, because the number new objects and internal copy operations is reduced drastically.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.