You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Gianluca Righetto (Jira)" <ji...@apache.org> on 2020/04/22 15:22:00 UTC
[jira] [Comment Edited] (CASSANDRA-15624) Avoid lazy initializing
shut down instances when trying to send them messages
[ https://issues.apache.org/jira/browse/CASSANDRA-15624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17089774#comment-17089774 ]
Gianluca Righetto edited comment on CASSANDRA-15624 at 4/22/20, 3:21 PM:
-------------------------------------------------------------------------
In talking with [~marcuse], he told me this is actually an improvement around the in-jvm tests and it's not currently breaking any tests or making them flakey.
I did some investigation and it seems that with the recent changes in CASSANDRA-15501 ([https://github.com/apache/cassandra/commit/16f639af94f56cdc7145299730278e27a9113e2e]), the in-jvm Wrapper instance will now check if the delegate is null and prevent any operations on a shutdown node. Instead of failing silently or throw an NPE, it will now throw an IllegalStateException telling that "Can't use shut down instances, delegate is null".
[~marcuse] Do you think there's anything else we need to tackle in this ticket? /cc [~ifesdjeen]
Either way, being an improvement ticket, I don't think this is a blocker for 4.0 at this point and we can probably move it out. Any objections?
was (Author: gianluca):
In talking with [~marcuse], he told me this is actually an improvement around the in-jvm tests and it's not currently breaking any tests or making them flakey.
I did some investigation and it seems that with the recent changes in CASSANDRA-15501 ([https://github.com/apache/cassandra/commit/16f639af94f56cdc7145299730278e27a9113e2e]), the in-jvm Wrapper instance will now check if the delegate is null and prevent any operations on a shutdown node. Instead of failing silently or throw an NPE, it will now throw an IllegalStateException telling that ""Can't use shut down instances, delegate is null".
[~marcuse] Do you think there's anything else we need to tackle in this ticket? /cc [~ifesdjeen]
Either way, being an improvement ticket, I don't think this is a blocker for 4.0 at this point and we can probably move it out. Any objections?
> Avoid lazy initializing shut down instances when trying to send them messages
> -----------------------------------------------------------------------------
>
> Key: CASSANDRA-15624
> URL: https://issues.apache.org/jira/browse/CASSANDRA-15624
> Project: Cassandra
> Issue Type: Bug
> Components: Test/dtest
> Reporter: Marcus Eriksson
> Assignee: Gianluca Righetto
> Priority: Normal
> Fix For: 4.0-alpha
>
>
> We currently use {{to.broadcastAddressAndPort()}} when figuring out if we should send a message to an instance, if that instance has been shut down it will get re-initialized but not startup:ed which makes the tests fail.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cassandra.apache.org
For additional commands, e-mail: commits-help@cassandra.apache.org