[
https://issues.apache.org/jira/browse/SOLR-17448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17895333#comment-17895333
]
ASF subversion and git services commented on SOLR-17448:
--------------------------------------------------------
Commit 4c211ba5be43c63d3b9a7ccf5810b4aa73960e80 in solr's branch
refs/heads/branch_9_7 from Sanjay Dutt
[ https://gitbox.apache.org/repos/asf?p=solr.git;h=4c211ba5be4 ]
SOLR-17448 SOLR-17497: IndexFetcher, catch exception instead of bubbling up
uncaught (#2800)
(cherry picked from commit cc30093c5ee988555389b50cf2333edf743bb50f)
> Audit usage of ExecutorService#submit in Solr codebase
> ------------------------------------------------------
>
> Key: SOLR-17448
> URL: https://issues.apache.org/jira/browse/SOLR-17448
> Project: Solr
> Issue Type: Improvement
> Affects Versions: 9.7
> Reporter: Andrey Bozhko
> Priority: Minor
> Labels: pull-request-available
> Fix For: 9.8
>
> Time Spent: 2.5h
> Remaining Estimate: 0h
>
> There are quite a few places in Solr codebase where the background task is
> created by invoking `ExecutorService#submit(...)` method - but where the
> reference to the returned future is not retained.
> So if the background task fails for any reason, and the task doesn't itself
> have a try-catch block to log the failure, - the failure will go completely
> unnoticed.
>
> This ticket is to review the usage of ExecutorService#submit method in the
> codebase, and replace those with Executor#execute where appropriate.
>
> Originally brought up in the dev mailing list:
> [https://lists.apache.org/thread/5f1965rltcspgw0j8nzcn2qnz9l4s8qm]
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org
