[
https://issues.apache.org/jira/browse/CASSANDRA-12618?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15506443#comment-15506443
]
Benjamin Lerer commented on CASSANDRA-12618:
--------------------------------------------
bq. Is there any other reason why you dont use it on sets and maps?
This is why you should not code when you are suffering from jetlag (I even
forgot that I had written that patch for a week ;-)).
Thanks, I will fix it straight away.
> Out of memory bug with one insert
> ----------------------------------
>
> Key: CASSANDRA-12618
> URL: https://issues.apache.org/jira/browse/CASSANDRA-12618
> Project: Cassandra
> Issue Type: Bug
> Reporter: Eduardo Alonso de Blas
> Assignee: Benjamin Lerer
> Priority: Critical
> Attachments: EvilQuery.tar.gz
>
>
> When executing an INSERT built by QueryBuilder in java driver produces an
> OutOfMemory in the server.
> Having a table with a List<String> field like this:
> CREATE TABLE keyspace_name.table_name(
> pk uuid,
> mylist list<text>,
> PRIMARY KEY (pk)
> );
> Anyone can build an INSERT with QueryBuilder like this:
> *Statement insert = QueryBuilder.insertInto(keyspace, table).value("pk",
> UUID.randomUUID()).value("mylist","blabla");*
> *session.execute(insert);*
> just trying to set a String where should be a List<String>.
> I have set tracing on in the cassandra node.
> TRACE [SharedPool-Worker-2] 2016-09-06 19:24:37,964 Message.java:506 -
> Received: QUERY INSERT INTO test_evil.table_aa (pk,mylist) VALUES
> (?,?);[pageSize = 5000], v=4
> DEBUG [SharedPool-Worker-2] 2016-09-06 19:24:37,967 ListSerializer.java:92 -
> deseiralizign a ByteBuffer in List lenght: 1651269986
> TRACE [EXPIRING-MAP-REAPER:1] 2016-09-06 19:24:38,319 ExpiringMap.java:102 -
> Expired 0 entries
> TRACE [SharedPool-Worker-2] 2016-09-06 19:24:38,320 Tracing.java:182 -
> request complete
> TRACE [Service Thread] 2016-09-06 19:24:38,321 GCInspector.java:286 - ParNew
> GC in 41ms. CMS Old Gen: 0 -> 9337048; Par Eden Space: 525391216 -> 0; Par
> Survivor Space: 18860168 -> 30843544
> TRACE [Service Thread] 2016-09-06 19:24:38,322 GCInspector.java:286 -
> ConcurrentMarkSweep GC in 148ms. CMS Old Gen: 9337048 -> 37098008; Par
> Survivor Space: 30843544 -> 0
> TRACE [Service Thread] 2016-09-06 19:24:38,322 GCInspector.java:286 -
> ConcurrentMarkSweep GC in 0ms. CMS Old Gen: 37098008 -> 36063280;
> ERROR [SharedPool-Worker-2] 2016-09-06 19:24:38,323
> JVMStabilityInspector.java:140 - JVM state determined to be unstable.
> Exiting forcefully due to:
> java.lang.OutOfMemoryError: Java heap space
> at java.util.ArrayList.<init>(ArrayList.java:152) ~[na:1.8.0_101]
> at
> org.apache.cassandra.serializers.ListSerializer.deserializeForNativeProtocol(ListSerializer.java:93)
> ~[main/:na]
> at org.apache.cassandra.cql3.Lists$Value.fromSerialized(Lists.java:137)
> ~[main/:na]
> at org.apache.cassandra.cql3.Lists$Marker.bind(Lists.java:242)
> ~[main/:na]
> at org.apache.cassandra.cql3.Lists$Setter.execute(Lists.java:295)
> ~[main/:na]
> at
> org.apache.cassandra.cql3.statements.UpdateStatement.addUpdateForKey(UpdateStatement.java:94)
> ~[main/:na]
> at
> org.apache.cassandra.cql3.statements.ModificationStatement.addUpdates(ModificationStatement.java:676)
> ~[main/:na]
> at
> org.apache.cassandra.cql3.statements.ModificationStatement.getMutations(ModificationStatement.java:616)
> ~[main/:na]
> at
> org.apache.cassandra.cql3.statements.ModificationStatement.executeWithoutCondition(ModificationStatement.java:429)
> ~[main/:na]
> at
> org.apache.cassandra.cql3.statements.ModificationStatement.execute(ModificationStatement.java:417)
> ~[main/:na]
> at
> com.stratio.cassandra.lucene.IndexQueryHandler.execute(IndexQueryHandler.java:181)
> ~[cassandra-lucene-index-plugin-3.7.2-RC1-SNAPSHOT.jar:na]
> at
> com.stratio.cassandra.lucene.IndexQueryHandler.processStatement(IndexQueryHandler.java:155)
> ~[cassandra-lucene-index-plugin-3.7.2-RC1-SNAPSHOT.jar:na]
> at
> com.stratio.cassandra.lucene.IndexQueryHandler.process(IndexQueryHandler.java:129)
> ~[cassandra-lucene-index-plugin-3.7.2-RC1-SNAPSHOT.jar:na]
> at
> org.apache.cassandra.transport.messages.QueryMessage.execute(QueryMessage.java:115)
> ~[main/:na]
> at
> org.apache.cassandra.transport.Message$Dispatcher.channelRead0(Message.java:507)
> [main/:na]
> at
> org.apache.cassandra.transport.Message$Dispatcher.channelRead0(Message.java:401)
> [main/:na]
> at
> io.netty.channel.SimpleChannelInboundHandler.channelRead(SimpleChannelInboundHandler.java:105)
> [netty-all-4.0.36.Final.jar:4.0.36.Final]
> at
> io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:292)
> [netty-all-4.0.36.Final.jar:4.0.36.Final]
> at
> io.netty.channel.AbstractChannelHandlerContext.access$600(AbstractChannelHandlerContext.java:32)
> [netty-all-4.0.36.Final.jar:4.0.36.Final]
> at
> io.netty.channel.AbstractChannelHandlerContext$7.run(AbstractChannelHandlerContext.java:283)
> [netty-all-4.0.36.Final.jar:4.0.36.Final]
> at
> java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
> [na:1.8.0_101]
> at
> org.apache.cassandra.concurrent.AbstractLocalAwareExecutorService$FutureTask.run(AbstractLocalAwareExecutorService.java:164)
> [main/:na]
> at org.apache.cassandra.concurrent.SEPWorker.run(SEPWorker.java:105)
> [main/:na]
> at java.lang.Thread.run(Thread.java:745) [na:1.8.0_101]
> I think that QueryBuilder builds the query with prepared statement: INSERT
> INTO test_evil.table_aa (pk,mylist) VALUES (?,?); and in the server side it
> does not validate the correct types.
> I have tested this with normal CQL in cqlsh and it works well.
> So, i think it is only in the prepared statement validation.
> I attached a maven project in order to help you to reproduce it.
> To generate the jar: 'mvn clean compile assembly:single'
> To execute it: 'java -jar
> target/EvilQuery-1.0-SNAPSHOT-jar-with-dependencies.jar -host localhost
> -keyspace keyspace_name -table table_name'
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
