You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by jb...@apache.org on 2009/10/20 02:28:12 UTC
svn commit: r826916 -
/incubator/cassandra/branches/cassandra-0.4/src/java/org/apache/cassandra/db/SuperColumn.java
Author: jbellis
Date: Tue Oct 20 00:28:11 2009
New Revision: 826916
URL: http://svn.apache.org/viewvc?rev=826916&view=rev
Log:
Fix using non-utf8-aware comparison as a sanity check in SC.putColumn. patch by jbellis; tested by Edmond Lau for CASSANDRA-493
Modified:
incubator/cassandra/branches/cassandra-0.4/src/java/org/apache/cassandra/db/SuperColumn.java
Modified: incubator/cassandra/branches/cassandra-0.4/src/java/org/apache/cassandra/db/SuperColumn.java
URL: http://svn.apache.org/viewvc/incubator/cassandra/branches/cassandra-0.4/src/java/org/apache/cassandra/db/SuperColumn.java?rev=826916&r1=826915&r2=826916&view=diff
==============================================================================
--- incubator/cassandra/branches/cassandra-0.4/src/java/org/apache/cassandra/db/SuperColumn.java (original)
+++ incubator/cassandra/branches/cassandra-0.4/src/java/org/apache/cassandra/db/SuperColumn.java Tue Oct 20 00:28:11 2009
@@ -201,14 +201,7 @@
*/
public void putColumn(IColumn column)
{
- if (!(column instanceof SuperColumn))
- {
- throw new UnsupportedOperationException("Only Super column objects should be put here");
- }
- if (!Arrays.equals(name_, column.name()))
- {
- throw new IllegalArgumentException("The name should match the name of the current column or super column");
- }
+ assert column instanceof SuperColumn;
for (IColumn subColumn : column.getSubColumns())
{