[ 
https://issues.apache.org/jira/browse/CASSANDRA-17049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17442781#comment-17442781
 ] 

Aleksey Yeschenko commented on CASSANDRA-17049:
-----------------------------------------------

Cheers, committed with the suggested change as 
[58c878e3e0356286ac1418e5e52e6f5cadd22ddb|https://github.com/apache/cassandra/commit/58c878e3e0356286ac1418e5e52e6f5cadd22ddb]
 and merged up. 

> Fix rare NPE caused by batchlog replay / node decomission races
> ---------------------------------------------------------------
>
>                 Key: CASSANDRA-17049
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-17049
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Consistency/Batch Log, Consistency/Hints
>            Reporter: Aleksey Yeschenko
>            Assignee: Aleksey Yeschenko
>            Priority: Low
>             Fix For: 3.0.x, 3.11.x, 4.0.x, 4.x
>
>
> Batchlog replay process collects addresses of the hosts that have been hinted 
> to, so it can flush hints for them to disk before confirming deletion of the 
> replayed batches. If a node has been decommissioned during replay, however, 
> when the time comes to flush the hints at the very end of replay, 
> {{StorageService.getHostIdForEndpoint()}} will return {{null}} for its 
> address, which will, down the line, cause {{HintsCatalog::get()}} to be 
> invoked with a {{null}} host id argument, causing an NPE.
> The simple fix is to check returned host ids for addresses for nulls, and 
> collect hinted host ids instead of hinted addresses.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

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

Reply via email to