[ https://issues.apache.org/jira/browse/CASSANDRA-16276?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17234647#comment-17234647 ]
Aleksey Yeschenko commented on CASSANDRA-16276: ----------------------------------------------- Cheers, committed as [7ca997ba3514e19864d53b8ca56a1e4f5c26208f|https://github.com/apache/cassandra/commit/7ca997ba3514e19864d53b8ca56a1e4f5c26208f] to trunk. > 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 > Fix For: 4.0-beta > > > 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: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org