[ https://issues.apache.org/jira/browse/CASSANDRA-18302 ]
Caleb Rackliffe deleted comment on CASSANDRA-18302:
---------------------------------------------
was (Author: maedhroz):
Looking at the update patch...
It seems like all we need to get the original statement are the arguments
passed to {{StatementSource.create()}}. (i.e. the start/end tokens and the
token stream) What if we just held those in the {{StatementSource}} until we
needed them in {{describe()}}, which I see is now being called lazily. In that
case, can't we avoid materializing those strings until we actually have to
report an error?
> Fix AIOOBE and improve validation messages for transaction statements
> ---------------------------------------------------------------------
>
> Key: CASSANDRA-18302
> URL: https://issues.apache.org/jira/browse/CASSANDRA-18302
> Project: Cassandra
> Issue Type: Improvement
> Components: Accord
> Reporter: Jacek Lewandowski
> Assignee: Jacek Lewandowski
> Priority: Normal
> Fix For: 5.x
>
> Time Spent: 2h
> Remaining Estimate: 0h
>
> Currently it happens sometimes that ArrayIndexOutOfBoundsException is thrown
> from asCql method when validation transaction statement. In addition, asCql
> does not return precisely the query user entered so the whole error message
> can be misleading.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org
