[
https://issues.apache.org/jira/browse/CASSANDRA-4242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13276563#comment-13276563
]
Sylvain Lebresne commented on CASSANDRA-4242:
---------------------------------------------
I can see the point of returning the names in the CqlPreparedResult. However,
despite Jonathan's suggestion, I actually prefered the original one (except for
the binary->string), i.e.:
{noformat}
struct CqlPreparedResult {
1: required i32 itemId,
2: required i32 count,
3: optional list<string> variable_types
+ 4: optional list<string> variable_names
}
{noformat}
The reason is mainly that I don't see the point in breaking CQL3 clients. I
agree CqlMetadata is overkill for CQL3, but imo CASSANDRA-2478 is where we
should bother fixing this.
Also, the variable_types in CqlPreparedResult is actually not a CQL2 thing, but
a CQL3 thing exclusively, so deprecating it in say 1.1.1 when it was introduced
in 1.1.0 feels again a bit harsh on the client writers for no good reasons.
bq. I have the feeling that C* is moving away from sliced query on binary
column names (a strong feature in my opinion)
It does not. The difference is that a column name *in CQL3* does not
necessarily map to a internal column name. So CQL3 column names *are* strings
and this without any loss of generality.
bq. Moreover, CqlMetadata exposes column names as binary
Yes, but that part is due to the difference between CQL2 and CQL3. Concretely,
CQL3 will *never* return anything else than UTF8 bytes (which is why Jonathan
suggested replacing CqlMetadata by a map<string, string> for CQL3, but see my
argument above). In any case, given that the variable names will be CQL3 only
(we cannot, even if we wanted to, do it in CQL2), I don't think there is a
point in returning binary.
> Name of parameters should be available in CqlPreparedResult
> -----------------------------------------------------------
>
> Key: CASSANDRA-4242
> URL: https://issues.apache.org/jira/browse/CASSANDRA-4242
> Project: Cassandra
> Issue Type: Improvement
> Components: Core
> Affects Versions: 1.1.0
> Reporter: Pierre Chalamet
> Priority: Minor
> Attachments: 4242.txt, 4242_2.txt, 4242_3.txt
>
>
> Client side, it could be nice to have the name of parameters in
> CqlPreparedResult. This could allow parameters mapping by name instead of by
> index.
> {code}
> struct CqlNameType {
> 1: required binary key,
> 2: required string type
> }
> struct CqlPreparedResult {
> 1: required i32 itemId,
> 2: required i32 count,
> 3: optional list<string> variable_types,
> 4: optional list<CqlNameType> name_types
> }
> {code}
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
