cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ryan McGuire (JIRA)" <j...@apache.org>
Subject [jira] [Assigned] (CASSANDRA-7109) Create replace_address dtest
Date Tue, 29 Apr 2014 20:57:23 GMT

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

Ryan McGuire reassigned CASSANDRA-7109:
---------------------------------------

    Assignee: Ryan McGuire  (was: Shawn Kumar)

> Create replace_address dtest
> ----------------------------
>
>                 Key: CASSANDRA-7109
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-7109
>             Project: Cassandra
>          Issue Type: Test
>            Reporter: Ryan McGuire
>            Assignee: Ryan McGuire
>
> {noformat}
> 16:03 < driftx> well, this just bothers me because either it's been broken for
almost ever, or something 
>                 broke in cassandra.
> 16:43 < thobbs> driftx: I'm testing your patch on #6622, but I'm seeing a bit of
a weird error:
> 16:43 < CassBotJr> https://issues.apache.org/jira/browse/CASSANDRA-6622 (Unresolved;
1.2.16, 2.0.6): 
>                    "Streaming session failures during node replace of same address"
> 16:43 < thobbs> java.lang.UnsupportedOperationException: Cannot replace token -1017822742317066613
which does 
>                 not exist!
> 16:44 < thobbs> this is on 2.0 with the patch applied
> 16:44 < driftx> O_o
> 16:44 < thobbs> I'm just stopping a ccm node, clearing it, then starting with replace_address
(auto_bootstrap 
>                 = true, not a seed, initial_tokens is null)
> 16:45 < driftx> oh, I'm stupid, hang on
> 16:47 < rcoli> is the sum of that that replace_* is still broken in 2.0 ?
> 16:47 < rcoli> err, 1.2?
> 16:48 < driftx> thobbs: updated the patch
> 16:48 < thobbs> rcoli: only for replacing the same address
> 16:48 < rcoli> is there another case?
> 16:49 < driftx> replacing with a different address.
> 16:49 < rcoli> oh, right, _address_
> 16:49 < rcoli> I'm still modeling this as replace _token_
> 16:49 < rcoli> in my brain
> 16:49 < driftx> same address never broke for me though, so you can probably just
retry
> 16:55 < thobbs> can we add a dtest for replace_address coverage?  It's kind of
annoying to test manually and 
>                 we've managed to break it a few times
> 16:56 < thobbs> I have a PR against ccm open to add replace_address support: 
>                 https://github.com/pcmanus/ccm/pull/85
> 16:57 < driftx> I could have sworn we had one
> 16:58 < driftx> we do but it's using replace_token so probably not even running
now
> 16:58 < thobbs> yeah
> 16:58 < thobbs> it would be nice to cover replacing the same address, another address,
and expected failures 
>                 like replacing a still-live node
> 16:59 < driftx> +1
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message