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/07/10 19:48:41 UTC
svn commit: r793050 - in /incubator/cassandra/trunk:
src/java/org/apache/cassandra/db/ReadCommand.java
src/java/org/apache/cassandra/db/RowReadCommand.java
test/unit/org/apache/cassandra/db/ReadMessageTest.java
Author: jbellis
Date: Fri Jul 10 17:48:41 2009
New Revision: 793050
URL: http://svn.apache.org/viewvc?rev=793050&view=rev
Log:
r/m unused (and dangerous) RowReadCommand.
patch by jbellis; reviewed by Jun Rao for CASSANDRA-287
Removed:
incubator/cassandra/trunk/src/java/org/apache/cassandra/db/RowReadCommand.java
Modified:
incubator/cassandra/trunk/src/java/org/apache/cassandra/db/ReadCommand.java
incubator/cassandra/trunk/test/unit/org/apache/cassandra/db/ReadMessageTest.java
Modified: incubator/cassandra/trunk/src/java/org/apache/cassandra/db/ReadCommand.java
URL: http://svn.apache.org/viewvc/incubator/cassandra/trunk/src/java/org/apache/cassandra/db/ReadCommand.java?rev=793050&r1=793049&r2=793050&view=diff
==============================================================================
--- incubator/cassandra/trunk/src/java/org/apache/cassandra/db/ReadCommand.java (original)
+++ incubator/cassandra/trunk/src/java/org/apache/cassandra/db/ReadCommand.java Fri Jul 10 17:48:41 2009
@@ -92,7 +92,6 @@
private static final Map<Byte, ReadCommandSerializer> CMD_SERIALIZER_MAP = new HashMap<Byte, ReadCommandSerializer>();
static
{
- CMD_SERIALIZER_MAP.put(ReadCommand.CMD_TYPE_GET_ROW, new RowReadCommandSerializer());
CMD_SERIALIZER_MAP.put(ReadCommand.CMD_TYPE_GET_COLUMN, new ColumnReadCommandSerializer());
CMD_SERIALIZER_MAP.put(ReadCommand.CMD_TYPE_GET_SLICE_BY_NAMES, new SliceByNamesReadCommandSerializer());
CMD_SERIALIZER_MAP.put(ReadCommand.CMD_TYPE_GET_COLUMNS_SINCE, new ColumnsSinceReadCommandSerializer());
Modified: incubator/cassandra/trunk/test/unit/org/apache/cassandra/db/ReadMessageTest.java
URL: http://svn.apache.org/viewvc/incubator/cassandra/trunk/test/unit/org/apache/cassandra/db/ReadMessageTest.java?rev=793050&r1=793049&r2=793050&view=diff
==============================================================================
--- incubator/cassandra/trunk/test/unit/org/apache/cassandra/db/ReadMessageTest.java (original)
+++ incubator/cassandra/trunk/test/unit/org/apache/cassandra/db/ReadMessageTest.java Fri Jul 10 17:48:41 2009
@@ -49,10 +49,6 @@
rm2 = serializeAndDeserializeReadMessage(rm);
assert rm2.toString().equals(rm.toString());
- rm = new RowReadCommand("Table1", "row1");
- rm2 = serializeAndDeserializeReadMessage(rm);
- assert rm2.toString().equals(rm.toString());
-
rm = new ColumnsSinceReadCommand("Table1", "row1", "foo", 1);
rm2 = serializeAndDeserializeReadMessage(rm);
assert rm2.toString().equals(rm.toString());