[
https://issues.apache.org/jira/browse/CASSANDRA-8375?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14353295#comment-14353295
]
Joshua McKenzie commented on CASSANDRA-8375:
--------------------------------------------
v3 isn't compiling for me:
{noformat}
home/jmckenzie/src/cassandra/test/unit/org/apache/cassandra/locator/OldNetworkTopologyStrategyTest.java:295:
error: cannot infer type arguments for Range<>;
[javac] toFetchExpected[0] = new
Range<>(getToken(movingNodeIdxAfterMove - 1, tokens),
getToken(movingNodeIdxAfterMove, tokens));
[javac] ^
[javac] reason: inferred type does not conform to declared bound(s)
[javac] inferred: BigIntegerToken
[javac] bound(s): RingPosition<BigIntegerToken>
{noformat}
> Cleanup of generics in bounds serialization
> -------------------------------------------
>
> Key: CASSANDRA-8375
> URL: https://issues.apache.org/jira/browse/CASSANDRA-8375
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Branimir Lambov
> Assignee: Branimir Lambov
> Priority: Trivial
> Fix For: 3.0
>
> Attachments: 8375-v2.patch, 8375-v3.patch, 8375.patch
>
>
> There is currently a single serializer for {{AbstractBounds}} applied to both
> {{Token}} and {{RowPosition}} ranges and bounds. This serializer does not
> know which kind of bounds it needs to work with, which causes some
> necessarily unsafe conversions and needs extra code in all bounds types
> ({{toRowBounds}}/{{toTokenBounds}}) to make the conversions safe, the
> application of which can be easily forgotten.
> As all users of this serialization know in advance the kind of range they
> want to serialize, this can be replaced by simpler type-specific
> serialization.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
