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

Aleksey Yeschenko updated CASSANDRA-16276:
------------------------------------------
    Description: 
If we invoke nodetool drain before shutdown, it sometimes fails to shut down 
messaging service in time (in this case - timing out the shutdown of the 
eventloopgroup by Netty). But, not before we manage to set isShutdown of 
StorageService to true, despite aborting further drain logic (including 
shutting down mutation stages).

Then, via on shutdown hook, we invoke drain() method again, implicitly. We see 
that the mutation stage is not shutdown and proceed to assert that isShutdown 
== false, failing that assertion and triggering a second error log message.

The patch merely ensures that any exception thrown by MS shutdown is captured 
so that drain logic can complete the first time around.

  was:TBA (trivial)


> Drain and/or shutdown might throw because of slow messaging service shutdown
> ----------------------------------------------------------------------------
>
>                 Key: CASSANDRA-16276
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-16276
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Local/Startup and Shutdown
>            Reporter: Aleksey Yeschenko
>            Assignee: Aleksey Yeschenko
>            Priority: Normal
>
> If we invoke nodetool drain before shutdown, it sometimes fails to shut down 
> messaging service in time (in this case - timing out the shutdown of the 
> eventloopgroup by Netty). But, not before we manage to set isShutdown of 
> StorageService to true, despite aborting further drain logic (including 
> shutting down mutation stages).
> Then, via on shutdown hook, we invoke drain() method again, implicitly. We 
> see that the mutation stage is not shutdown and proceed to assert that 
> isShutdown == false, failing that assertion and triggering a second error log 
> message.
> The patch merely ensures that any exception thrown by MS shutdown is captured 
> so that drain logic can complete the first time around.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to