[ 
https://issues.apache.org/jira/browse/SOLR-7956?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Scott Blum updated SOLR-7956:
-----------------------------
    Attachment: SOLR-7956-commit-tracker.patch

Some of those might be okay, I don't know the code well enough.  However, the 
shutdownNow() call in CommitTracker is definitely fatal.  I've attached a patch 
to demonstrate with a test case.

FWIW, the pending.cancel(true), while dangerous, is far less likely to be hit 
because the task nulls out the pending field as soon as it starts to run.  But 
the shutdownNow() will interrupt the thread even after the field is nulled out, 
so it's a lot more likely to really hit.

> There are interrupts on shutdown in places that can cause 
> ChannelAlreadyClosed exceptions which prevents proper closing of transaction 
> logs.
> --------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: SOLR-7956
>                 URL: https://issues.apache.org/jira/browse/SOLR-7956
>             Project: Solr
>          Issue Type: Bug
>            Reporter: Mark Miller
>            Assignee: Mark Miller
>             Fix For: Trunk, 5.4
>
>         Attachments: SOLR-7956-commit-tracker.patch, SOLR-7956.patch, 
> SOLR-7956.patch, SOLR-7956.patch
>
>
> Found this while beast testing HttpPartitionTest.



--
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