You are viewing a plain text version of this content. The canonical link for it is here.
Posted to solr-dev@lucene.apache.org by "Shalin Shekhar Mangar (JIRA)" <ji...@apache.org> on 2009/05/09 10:18:45 UTC

[jira] Commented: (SOLR-1157) Change for timeouts in RequestHandlerBase (rev674249) broke unit tests due to rsp.getResponse() returning null

    [ https://issues.apache.org/jira/browse/SOLR-1157?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12707641#action_12707641 ] 

Shalin Shekhar Mangar commented on SOLR-1157:
---------------------------------------------

Which unit test was this?

> Change for timeouts in RequestHandlerBase (rev674249) broke unit tests due to rsp.getResponse() returning null
> --------------------------------------------------------------------------------------------------------------
>
>                 Key: SOLR-1157
>                 URL: https://issues.apache.org/jira/browse/SOLR-1157
>             Project: Solr
>          Issue Type: Bug
>          Components: search
>    Affects Versions: 1.4
>            Reporter: Jayson Minard
>            Priority: Critical
>
> This code:
> <code>
> boolean timedOut = (Boolean) rsp.getResponseHeader().get("partialResults") == null ? false : (Boolean) rsp.getResponseHeader().get("partialResults");
>         if (timedOut) {
>           numTimeouts++;
>           rsp.setHttpCaching(false);
>         }
> </code>
> in RequestHandlerBase around line 128 breaks unit tests.  Needs a null check on getResponseHeader() before going further.

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