[
https://issues.apache.org/jira/browse/SOLR-12477?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16561385#comment-16561385
]
jefferyyuan commented on SOLR-12477:
------------------------------------
[~varunthacker]
It makes sense(as Mockito doesn't work with newer java) , and I have reverted
the change at DirectUpdateHandlerTest#testAddDocThrowAlreadyClosedException.
Please check and thanks.
> Return server error(500) for AlreadyClosedException instead of client
> Errors(400)
> ---------------------------------------------------------------------------------
>
> Key: SOLR-12477
> URL: https://issues.apache.org/jira/browse/SOLR-12477
> Project: Solr
> Issue Type: Improvement
> Security Level: Public(Default Security Level. Issues are Public)
> Components: update
> Reporter: jefferyyuan
> Assignee: Varun Thacker
> Priority: Minor
> Labels: update
> Fix For: 7.3.2, master (8.0)
>
> Attachments: SOLR-12477.patch
>
> Time Spent: 40m
> Remaining Estimate: 0h
>
> In some cases(for example: corrupt index), addDoc0 throws
> AlreadyClosedException, but solr server returns client error 400 to client
> This will confuse customers and especially monitoring tool.
> Patch: [https://github.com/apache/lucene-solr/pull/402]
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org
