You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Jonathan Ellis (JIRA)" <ji...@apache.org> on 2009/04/07 22:39:12 UTC
[jira] Created: (CASSANDRA-65) Support for non-hash based
partitioners
Support for non-hash based partitioners
---------------------------------------
Key: CASSANDRA-65
URL: https://issues.apache.org/jira/browse/CASSANDRA-65
Project: Cassandra
Issue Type: Improvement
Reporter: Jonathan Ellis
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jonathan Ellis updated CASSANDRA-65:
------------------------------------
Attachment: (was: 0005-r-m-StorageService.token-in-favor-of-explicitly-passin.txt)
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jun Rao (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12697320#action_12697320 ]
Jun Rao commented on CASSANDRA-65:
----------------------------------
I looked at the patch. A few comments.
1. I'd leave ServiceStorage.showTheRing() there. It seems to be useful for some future utility.
2. Rename IPartitioner.getDefaultToken to getRandomToken, since it's not really the expected default behavior.
3. Why use elipsis instead of array in the following function?
public BootStrapper(EndPoint[] target, Token... token)
4. I'd leave SystemTable.main for future unit test.
5. This may be for future improvement. The new orderpreserving partitioner still has the problem that the cluster may not be balanced for arbitrary keys. One way to fix this is to let applications specify the initial token for each node. Then we need a way to initialize token from the conf file.
Other than those, the patch looks fine and ant test passes. However, since this is a relative major change, I think either Avinash or Prashant should take another look at it.
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt, 0005-r-m-StorageService.token-in-favor-of-explicitly-pass.patch, 0006-add-test-for-ReplicaPlacementStrategy-covering-both.patch
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jonathan Ellis updated CASSANDRA-65:
------------------------------------
Attachment: 0005-r-m-StorageService.token-in-favor-of-explicitly-passin.txt
0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt
0003-clean-up-partition-strategies-in-particular-implemen.txt
0002-r-m-unused-code.txt
0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt, 0005-r-m-StorageService.token-in-favor-of-explicitly-passin.txt
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jonathan Ellis updated CASSANDRA-65:
------------------------------------
Attachment: 65.tgz
Rebase against current HEAD. Added changes requested by Jun.
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Avinash Lakshman
> Fix For: 0.3
>
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt, 0005-r-m-StorageService.token-in-favor-of-explicitly-pass.patch, 0006-add-test-for-ReplicaPlacementStrategy-covering-both.patch, 65.tgz
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12697086#action_12697086 ]
Jonathan Ellis commented on CASSANDRA-65:
-----------------------------------------
removed 0005 until the test is finished.
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Hudson (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12703641#action_12703641 ]
Hudson commented on CASSANDRA-65:
---------------------------------
Integrated in Cassandra #51 (See [http://hudson.zones.apache.org/hudson/job/Cassandra/51/])
rename getDefaultToken -> getInitialToken
patch by jbellis; reviewed by Jun Rao for
add test for ReplicaPlacementStrategy covering both Random and OrderPreserving partitioners
patch by jbellis; reviewed by Jun Rao for
r/m StorageService.token in favor of explicitly passing a Partitioner object. this allows testing of components independent of the static SS.
patch by jbellis; reviewed by Jun Rao for
replace BigInteger tokens with BigIntegerToken and StringToken in RandomPartitioner and OrderPreservingPartitioner, respectively. (OrderPreservingHashPartitioner is no more.)
Doing order preserving partitioning based on the raw string has a number of compelling advantages:
- there is no key length that all tokens must be padded to (which can be expensive if there is a range of key lengths) and which cannot be increased after deployment
- it allows user-defined collations [sorting] rather than being limited to sorting by code point value (which is useless in the unicode world, and not always what you want even for ascii keys)
- it will work with all UTF-16 characters, not just the UCS-2 subset (this is a limitation of using as a base 2**16, i.e., assuming for your order preserving hash that all characters are two bytes).
patch by jbellis; reviewed by Jun Rao for
clean up partition strategies; in particular, implement getStorageEndPoints(String[] keys) in terms of getStorageEndPoints(BigInteger token), instead of copy/pasting code
patch by jbellis; reviewed by Jun Rao for
r/m unused code
patch by jbellis; reviewed by Jun Rao for
move *Partitioner from cassandra.service to cassandra.dht.
patch by jbellis; reviewed by Jun Rao for
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Fix For: 0.3
>
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt, 0005-r-m-StorageService.token-in-favor-of-explicitly-pass.patch, 0006-add-test-for-ReplicaPlacementStrategy-covering-both.patch, 65-v2.tgz, 65-v3.tgz, 65.tgz
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jonathan Ellis updated CASSANDRA-65:
------------------------------------
Attachment: 0005-v2.patch
oops, 0005 has errors. corrected version attached.
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt, 0005-r-m-StorageService.token-in-favor-of-explicitly-passin.txt, 0005-v2.patch
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12697033#action_12697033 ]
Jonathan Ellis commented on CASSANDRA-65:
-----------------------------------------
5 r/m StorageService.token in favor of explicitly passing a Partitioner object. this allows testing of components independent of the static SS.
4 replace BigInteger tokens with BigIntegerToken and StringToken in RandomPartitioner and OrderPreservingPartitioner, respectively. (OrderPreservingHashPartitioner is no more.)
Doing order preserving partitioning based on the raw string has a number of compellingadvantages:
- there is no key length that all tokens must be padded to (which can be expensive ifthere is a range of key lengths) and which cannot be increased after deployment
- it allows user-defined collations [sorting] rather than being limited to sorting bycode point value (which is useless in the unicode world, and not always what you want evenfor ascii keys)
- it will work with all UTF-16 characters, not just the UCS-2 subset (this is a limitation of using as a base 2**16, i.e., assuming for your order preserving hash that all characters are two bytes).
3 clean up partition strategies; in particular, implement getStorageEndPoints(String[] keys) in terms of getStorageEndPoints(BigInteger token), instead of copy/pasting code
2 r/m unused code
1 move *Partitioner from cassandra.service to cassandra.dht
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt, 0005-r-m-StorageService.token-in-favor-of-explicitly-passin.txt
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jonathan Ellis updated CASSANDRA-65:
------------------------------------
Attachment: 0006-add-test-for-ReplicaPlacementStrategy-covering-both.patch
Attached strategy cleanup + test patches.
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt, 0005-r-m-StorageService.token-in-favor-of-explicitly-pass.patch, 0006-add-test-for-ReplicaPlacementStrategy-covering-both.patch
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jonathan Ellis updated CASSANDRA-65:
------------------------------------
Attachment: 65-v2.tgz
rebase again.
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Avinash Lakshman
> Fix For: 0.3
>
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt, 0005-r-m-StorageService.token-in-favor-of-explicitly-pass.patch, 0006-add-test-for-ReplicaPlacementStrategy-covering-both.patch, 65-v2.tgz, 65.tgz
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jun Rao (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12697337#action_12697337 ]
Jun Rao commented on CASSANDRA-65:
----------------------------------
Another thing, I am not sure what the implication of changing BigInteger.ZERO.toString() to "" in StorageService.getToken() is. Did you verify?
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt, 0005-r-m-StorageService.token-in-favor-of-explicitly-pass.patch, 0006-add-test-for-ReplicaPlacementStrategy-covering-both.patch
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jonathan Ellis updated CASSANDRA-65:
------------------------------------
Fix Version/s: 0.3
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Avinash Lakshman
> Fix For: 0.3
>
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt, 0005-r-m-StorageService.token-in-favor-of-explicitly-pass.patch, 0006-add-test-for-ReplicaPlacementStrategy-covering-both.patch
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Assigned: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jonathan Ellis reassigned CASSANDRA-65:
---------------------------------------
Assignee: Avinash Lakshman (was: Jonathan Ellis)
> since this is a relative major change, I think either Avinash or Prashant should take another look at it.
Yes, and Avinash said he wanted to review it. Assigning to him.
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Avinash Lakshman
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt, 0005-r-m-StorageService.token-in-favor-of-explicitly-pass.patch, 0006-add-test-for-ReplicaPlacementStrategy-covering-both.patch
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jonathan Ellis updated CASSANDRA-65:
------------------------------------
Attachment: (was: 0005-v3.patch)
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Assigned: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jonathan Ellis reassigned CASSANDRA-65:
---------------------------------------
Assignee: Jonathan Ellis
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12697355#action_12697355 ]
Jonathan Ellis commented on CASSANDRA-65:
-----------------------------------------
Argh, Jira let me throw away my reply when I clicked on a link to doublecheck something. :( Here goes again.
1. If we need it later we can resurrect it from svn. Dead code is more confusing than useful.
2. We probably need a better name here -- the idea is "generate a token that can be assigned as the default for this node's endpoint if it starts up without having one configured." How about getInitialToken?
3. It just allows passing a single Token w/o explicitly wrapping it in an array (the compiler does it for you). IIRC I made use of this somewhere.
4. It's already in a unit test, in SystemTableTest. My bad for not removing the main() method earlier.
5. Yes, that will be in another patch.
6. Good question -- that method is only used by the status tool. So the question is what value to display when there is no token matching the endpoint. "0" is potentially confusing since it's a legitimate token value. Empty string leaves no doubt that there is no token. I can add a comment to this effect.
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt, 0005-r-m-StorageService.token-in-favor-of-explicitly-pass.patch, 0006-add-test-for-ReplicaPlacementStrategy-covering-both.patch
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jonathan Ellis updated CASSANDRA-65:
------------------------------------
Attachment: 0005-v3.patch
Got a bit farther into writing a test and found more static code I needed to clean out in 0005.
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt, 0005-r-m-StorageService.token-in-favor-of-explicitly-passin.txt, 0005-v2.patch, 0005-v3.patch
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jonathan Ellis updated CASSANDRA-65:
------------------------------------
Attachment: 65-v3.tgz
rebased, again
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Avinash Lakshman
> Fix For: 0.3
>
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt, 0005-r-m-StorageService.token-in-favor-of-explicitly-pass.patch, 0006-add-test-for-ReplicaPlacementStrategy-covering-both.patch, 65-v2.tgz, 65-v3.tgz, 65.tgz
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jonathan Ellis updated CASSANDRA-65:
------------------------------------
Attachment: 0005-r-m-StorageService.token-in-favor-of-explicitly-pass.patch
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt, 0005-r-m-StorageService.token-in-favor-of-explicitly-pass.patch
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Resolved: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jonathan Ellis resolved CASSANDRA-65.
-------------------------------------
Resolution: Fixed
Assignee: Jonathan Ellis (was: Avinash Lakshman)
It's been three weeks and no other reviews have been forthcoming. Committing.
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Fix For: 0.3
>
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt, 0005-r-m-StorageService.token-in-favor-of-explicitly-pass.patch, 0006-add-test-for-ReplicaPlacementStrategy-covering-both.patch, 65-v2.tgz, 65-v3.tgz, 65.tgz
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (CASSANDRA-65) Support for non-hash based
partitioners
Posted by "Jonathan Ellis (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CASSANDRA-65?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jonathan Ellis updated CASSANDRA-65:
------------------------------------
Attachment: (was: 0005-v2.patch)
> Support for non-hash based partitioners
> ---------------------------------------
>
> Key: CASSANDRA-65
> URL: https://issues.apache.org/jira/browse/CASSANDRA-65
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Jonathan Ellis
> Assignee: Jonathan Ellis
> Attachments: 0001-CASSANDRA-65-move-Partitioner-from-cassandra.service.txt, 0002-r-m-unused-code.txt, 0003-clean-up-partition-strategies-in-particular-implemen.txt, 0004-replace-BigInteger-tokens-with-BigIntegerToken-and-Str.txt
>
>
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.