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

Joel Bernstein edited comment on SOLR-9128 at 5/18/16 3:19 PM:
---------------------------------------------------------------

Ran across a few more issues while fixing error handling issues. So I'll bundle 
those fixes into this ticket as well:

FacetStream: was throwing NPE with zero facet results. TODO: add a test case 
for this. This effects the SQLHandler operating in facet aggregationMode.

GatherNodesStream: explain was breaking on the admin stream console, due to 
issues with toExpression.

CloudSolrStream: was throwing NPE on missing params rather then throwing proper 
exception. TODO: add test case for this.

GraphHandler: did not have the SelectStream registered.

Overall though the changes made in SOLR-8467 are looking good and the new admin 
stream console is working well.




was (Author: joel.bernstein):
Ran across a few more issues while fixing error handling issues. So I'll bundle 
those fixes into this ticket as well:

FacetStream: was throwing NPE with zero facet results. TODO: add a test case 
for this. This effects the SQLHandler operating in facet aggregationMode.

GatherNodesStream: explain was breaking on the admin stream console, due to 
issues with toExpression.

CloudSolrStream: was throwing NPE on missing params rather then throwing proper 
exception. TODOL add test case for this.

GraphHandler: did not have the SelectStream registered.



> Fix error handling issues in Streaming classes
> ----------------------------------------------
>
>                 Key: SOLR-9128
>                 URL: https://issues.apache.org/jira/browse/SOLR-9128
>             Project: Solr
>          Issue Type: Bug
>            Reporter: Joel Bernstein
>
> SOLR-8467 caused some error handling problems. This ticket will resolve those 
> issues.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to