cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Branimir Lambov (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (CASSANDRA-8244) Token, DecoratedKey, RowPosition and all bound types should not make any hidden references to the database partitioner
Date Fri, 21 Nov 2014 14:22:34 GMT

     [ 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)

Mime
View raw message