cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Ellis (JIRA)" <>
Subject [jira] [Commented] (CASSANDRA-4227) StorageProxy throws NPEs for when there's no hostids for a target
Date Wed, 23 May 2012 03:24:40 GMT


Jonathan Ellis commented on CASSANDRA-4227:

Sorry for the confusion, I was +1ing the 2nd patch.

I'm okay with NPEing if we have a null host on version 12, since that's Not Supposed To Happen.
 (Alternatively, we could add an assert to make that explicit.)
> StorageProxy throws NPEs for when there's no hostids for a target
> -----------------------------------------------------------------
>                 Key: CASSANDRA-4227
>                 URL:
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Core
>            Reporter: Dave Brosius
>            Priority: Trivial
>             Fix For: 1.2
>         Attachments: 4227_drop_hints.txt, 4227_guard_against_npes_for_old_gossip_versions.diff
> On trunk...
> if there is no host id due to an old node, an info log is generated, but the code continues
to use the null host id causing NPEs in decompose... Should this bypass this code, or perhaps
can the plain ip address be used in this case? don't know.
> as follows...
>                     UUID hostId = StorageService.instance.getTokenMetadata().getHostId(target);
>                     if ((hostId == null) && (Gossiper.instance.getVersion(target)
< MessagingService.VERSION_12))
>               "Unable to store hint for host with missing ID, {}
(old node?)", target.toString());
>                     RowMutation hintedMutation = RowMutation.hintFor(mutation, ByteBuffer.wrap(UUIDGen.decompose(hostId)));
>                     hintedMutation.apply();

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:!default.jspa
For more information on JIRA, see:


View raw message