You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@nutch.apache.org by "Julien Nioche (JIRA)" <ji...@apache.org> on 2009/09/29 12:40:16 UTC

[jira] Created: (NUTCH-756) CrawlDatum.set() does not resets Metadata if it is null

CrawlDatum.set() does not resets Metadata if it is null
-------------------------------------------------------

                 Key: NUTCH-756
                 URL: https://issues.apache.org/jira/browse/NUTCH-756
             Project: Nutch
          Issue Type: Bug
            Reporter: Julien Nioche
            Priority: Blocker


The patch Nutch-702 implemented the lazy instanciation of CrawlDatum objects, but when using the method set(CrawlDatum) to copy the content of an instance into the current object we did not reset the metadata of the current object if it was null in the instance passed as argument. As a result, the metadata of the current object might be kept from a previous CrawlDatum and won't correspond to the other fields of the instance.
The patch attached fixes this issue.

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


[jira] Commented: (NUTCH-756) CrawlDatum.set() does not reset Metadata if it is null

Posted by "Andrzej Bialecki (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/NUTCH-756?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12764028#action_12764028 ] 

Andrzej Bialecki  commented on NUTCH-756:
-----------------------------------------

Fixed in rev. 823557 - thanks!

> CrawlDatum.set() does not reset Metadata if it is null
> ------------------------------------------------------
>
>                 Key: NUTCH-756
>                 URL: https://issues.apache.org/jira/browse/NUTCH-756
>             Project: Nutch
>          Issue Type: Bug
>            Reporter: Julien Nioche
>            Assignee: Andrzej Bialecki 
>            Priority: Blocker
>             Fix For: 1.1
>
>         Attachments: NUTCH-756.patch
>
>
> The patch Nutch-702 implemented the lazy instanciation of CrawlDatum objects, but when using the method set(CrawlDatum) to copy the content of an instance into the current object we did not reset the metadata of the current object if it was null in the instance passed as argument. As a result, the metadata of the current object might be kept from a previous CrawlDatum and won't correspond to the other fields of the instance.
> The patch attached fixes this issue.

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


[jira] Commented: (NUTCH-756) CrawlDatum.set() does not reset Metadata if it is null

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/NUTCH-756?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12764297#action_12764297 ] 

Hudson commented on NUTCH-756:
------------------------------

Integrated in Nutch-trunk #959 (See [http://hudson.zones.apache.org/hudson/job/Nutch-trunk/959/])
     CrawlDatum.set() does not reset Metadata if it is null.


> CrawlDatum.set() does not reset Metadata if it is null
> ------------------------------------------------------
>
>                 Key: NUTCH-756
>                 URL: https://issues.apache.org/jira/browse/NUTCH-756
>             Project: Nutch
>          Issue Type: Bug
>            Reporter: Julien Nioche
>            Assignee: Andrzej Bialecki 
>            Priority: Blocker
>             Fix For: 1.1
>
>         Attachments: NUTCH-756.patch
>
>
> The patch Nutch-702 implemented the lazy instanciation of CrawlDatum objects, but when using the method set(CrawlDatum) to copy the content of an instance into the current object we did not reset the metadata of the current object if it was null in the instance passed as argument. As a result, the metadata of the current object might be kept from a previous CrawlDatum and won't correspond to the other fields of the instance.
> The patch attached fixes this issue.

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


[jira] Closed: (NUTCH-756) CrawlDatum.set() does not reset Metadata if it is null

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

Andrzej Bialecki  closed NUTCH-756.
-----------------------------------

       Resolution: Fixed
    Fix Version/s: 1.1
         Assignee: Andrzej Bialecki 

> CrawlDatum.set() does not reset Metadata if it is null
> ------------------------------------------------------
>
>                 Key: NUTCH-756
>                 URL: https://issues.apache.org/jira/browse/NUTCH-756
>             Project: Nutch
>          Issue Type: Bug
>            Reporter: Julien Nioche
>            Assignee: Andrzej Bialecki 
>            Priority: Blocker
>             Fix For: 1.1
>
>         Attachments: NUTCH-756.patch
>
>
> The patch Nutch-702 implemented the lazy instanciation of CrawlDatum objects, but when using the method set(CrawlDatum) to copy the content of an instance into the current object we did not reset the metadata of the current object if it was null in the instance passed as argument. As a result, the metadata of the current object might be kept from a previous CrawlDatum and won't correspond to the other fields of the instance.
> The patch attached fixes this issue.

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


[jira] Updated: (NUTCH-756) CrawlDatum.set() does not reset Metadata if it is null

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

Julien Nioche updated NUTCH-756:
--------------------------------

    Summary: CrawlDatum.set() does not reset Metadata if it is null  (was: CrawlDatum.set() does not resets Metadata if it is null)

> CrawlDatum.set() does not reset Metadata if it is null
> ------------------------------------------------------
>
>                 Key: NUTCH-756
>                 URL: https://issues.apache.org/jira/browse/NUTCH-756
>             Project: Nutch
>          Issue Type: Bug
>            Reporter: Julien Nioche
>            Priority: Blocker
>         Attachments: NUTCH-756.patch
>
>
> The patch Nutch-702 implemented the lazy instanciation of CrawlDatum objects, but when using the method set(CrawlDatum) to copy the content of an instance into the current object we did not reset the metadata of the current object if it was null in the instance passed as argument. As a result, the metadata of the current object might be kept from a previous CrawlDatum and won't correspond to the other fields of the instance.
> The patch attached fixes this issue.

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


[jira] Updated: (NUTCH-756) CrawlDatum.set() does not resets Metadata if it is null

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

Julien Nioche updated NUTCH-756:
--------------------------------

    Attachment: NUTCH-756.patch

Fixes issue with metadata not being properly overridden for CrawlDatum when calling the set() method

> CrawlDatum.set() does not resets Metadata if it is null
> -------------------------------------------------------
>
>                 Key: NUTCH-756
>                 URL: https://issues.apache.org/jira/browse/NUTCH-756
>             Project: Nutch
>          Issue Type: Bug
>            Reporter: Julien Nioche
>            Priority: Blocker
>         Attachments: NUTCH-756.patch
>
>
> The patch Nutch-702 implemented the lazy instanciation of CrawlDatum objects, but when using the method set(CrawlDatum) to copy the content of an instance into the current object we did not reset the metadata of the current object if it was null in the instance passed as argument. As a result, the metadata of the current object might be kept from a previous CrawlDatum and won't correspond to the other fields of the instance.
> The patch attached fixes this issue.

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