[
https://issues.apache.org/jira/browse/CASSANDRA-8244?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Branimir Lambov updated CASSANDRA-8244:
---------------------------------------
Attachment: 8244.patch
Rebased new version and patch are uploaded, comparison [at the same
location|https://github.com/blambov/cassandra/compare/8244-partitioner-in-token].
bq. getHeapSize on BigIntegerToken has a 'TODO: Probably wrong' comment for
getHeapSize()
Sorry, that wasn't to be in the final code. Since the hashes are all fixed-size
numbers, the heap size actually is correct, just badly named. Fixed static
field name.
bq. annotate @VisibleForTesting on
RandomPartitioner.BigIntegerToken.BigIntegerToken(String token)
Done.
bq. clean up import order on FBUtilities.java
Done.
bq. looks like changes to a couple of the .db files under test snuck in on
commit
Done.
> Token, DecoratedKey, RowPosition and all bound types should not make any
> hidden references to the database partitioner
> ----------------------------------------------------------------------------------------------------------------------
>
> Key: CASSANDRA-8244
> URL: https://issues.apache.org/jira/browse/CASSANDRA-8244
> Project: Cassandra
> Issue Type: Bug
> Reporter: Branimir Lambov
> Assignee: Branimir Lambov
> Priority: Minor
> Fix For: 3.0
>
> Attachments: 8244.patch
>
>
> Currently some of the functionality of Token refers to
> StorageService.getPartitioner() to avoid needing an extra argument. This is
> in turn implicitly used by RowPosition and then Range, causing possible
> problems, for example when ranges on secondary indices are used in a
> murmur-partitioned database.
> These references should be removed to force explicit choice of partitioner by
> callers; alternatively, the Token interface could be changed to provide a
> reference to the partitioner that created it.
> (Note: the hidden reference to partitioner in serialization is a separate
> issue.)
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
