[ 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