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

Hoss Man commented on SOLR-9842:
--------------------------------

+1 with 2 questions for mark...

# why Closable and not AutoClosable?
# can you think of any reason not to deprecate {{finish()}} an remove it from 
master? (moving all logic in the current finish impls into new {{close()}} 
methods?)

> UpdateRequestProcessors have no way to guarantee the closing of resources 
> used for a request.
> ---------------------------------------------------------------------------------------------
>
>                 Key: SOLR-9842
>                 URL: https://issues.apache.org/jira/browse/SOLR-9842
>             Project: Solr
>          Issue Type: Improvement
>      Security Level: Public(Default Security Level. Issues are Public) 
>            Reporter: Mark Miller
>            Assignee: Mark Miller
>         Attachments: SOLR-9842.patch
>
>
> You cannot count on finish being called for an UpdateRequestProcessor because 
> whether or not it's called in the face of errors is dependent on the other 
> processors in the chain.
> We should make UpdateRequestProcessor closeable and implement this so that 
> close is guaranteed to be called. I'm playing around with this in a patch for 
> another issue.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to