[ 
https://issues.apache.org/jira/browse/NUTCH-1722?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13889850#comment-13889850
 ] 

Daniel Kugel commented on NUTCH-1722:
-------------------------------------

Perhaps a warning should be added to the log in case this behavior was not 
expected because fetching the entire webpage table sounds like unwanted 
behavior. Actually perhaps a null batchid should fetch nothing and if one 
wishes to fetch everything he should specifically pass Nutch.ALL_BATCH_ID_STR?
I think that if null was passed by mistake its something the user would like to 
know.

> FetcherJob#fetch throws NullPointerException for null batchId
> -------------------------------------------------------------
>
>                 Key: NUTCH-1722
>                 URL: https://issues.apache.org/jira/browse/NUTCH-1722
>             Project: Nutch
>          Issue Type: Bug
>          Components: fetcher
>    Affects Versions: 2.2.1
>            Reporter: Gerhard Gossen
>         Attachments: NUTCH-1722-2.x.patch
>
>
> The documentation of FetcherJob#fetch claims that a null batchId parameter is 
> interpreted as "fetch all generated fetchlists". However, the implementation 
> tries to do {{batchId.equals(Nutch.ALL_BATCH_ID_STR)}}, which throws a 
> NullPointerException when batchId is null.
> The provided patch sets a null batchId to {{Nutch.ALL_BATCH_ID_STR}}, which 
> should be the documented behavior.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to