You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Sylvain Lebresne (JIRA)" <ji...@apache.org> on 2013/12/16 11:15:10 UTC

[jira] [Resolved] (CASSANDRA-6453) Improve error message for invalid property values during parsing.

     [ https://issues.apache.org/jira/browse/CASSANDRA-6453?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sylvain Lebresne resolved CASSANDRA-6453.
-----------------------------------------

    Resolution: Fixed
      Reviewer: Sylvain Lebresne

I understand that Brian's underlying problem was that he wants prepared statements for DDL queries which we indeed don't support.

But pragmatically, as far as this ticket description and patch goes, I don't see the harm in committing the error message improvement. It does is nicer to include the name of the property for which the value is invalid irregardless of the bind marker problem. Besides, Brian's confusion suggests that maybe an error message that explicitly indicate that bind markers are not supported would help too. So committed the patch with a slight specialization in the case of bind markers.


> Improve error message for invalid property values during parsing.
> -----------------------------------------------------------------
>
>                 Key: CASSANDRA-6453
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-6453
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Brian ONeill
>            Priority: Trivial
>         Attachments: CASSANDRA-6354-patch.txt
>
>
> Trivial change to the error message returned for invalid property values.
> Previously, it would just say "Invalid property value : ?".  If you were constructing a large prepared statement, with multiple question marks, it was difficult to track down which one the server was complaining about.  This enhancement tells you which one. =)



--
This message was sent by Atlassian JIRA
(v6.1.4#6159)