cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "paul cannon (JIRA)" <>
Subject [jira] [Commented] (CASSANDRA-4497) Update CQL pseudo-maps to real maps
Date Sat, 01 Sep 2012 03:13:07 GMT


paul cannon commented on CASSANDRA-4497:

bq. How is it more complicated to document that replication_parameters is a map like any other,
than replication_parameters has its own one-off syntax that only exists for historical reasons?
I don't buy it.

It's not, but it is more complicated when we need to say "if you're using CQL 3 with Cassandra
1.0 or 1.1, then you need to do this. If you're using CQL 3 with 1.2 or higher, then you need
to do this totally different thing. If you're using CQL 2, then you do something else." I
know, ideally we would have fully separate documentation sets and fully separate cqlsh versions
per major Cassandra version, but people don't seem to be very good at referring only to the
right one.

bq. reduce the special case weirdnesses exposed to users, as above

I'm admittedly thick, but which weirdnesses are reduced here?

bq. reduce special case weirdnesses in our internal implementation (which is also exposed
to "power users" in the system KS). so much more pleasant to "select replication_options from
schema_keyspaces where keyspace_name = 'foo'" and get a Map back.

I agree with that 100%, but this patch doesn't do that. It only changes the dedicated syntax
for setting keyspace and columnfamily options. I'd much prefer getting real maps inside the
system.schema* tables and leaving the cql syntax alone when we can. It has been a real moving
target, and swiping out the "first command you do" from under everyone's feet seems like it
might be discouraging.
> Update CQL pseudo-maps to real maps
> -----------------------------------
>                 Key: CASSANDRA-4497
>                 URL:
>             Project: Cassandra
>          Issue Type: Bug
>    Affects Versions: 1.2.0 beta 1
>            Reporter: Jonathan Ellis
>            Assignee: Pavel Yaskevich
>              Labels: cql3
>             Fix For: 1.2.0 beta 1
>         Attachments: CASSANDRA-4497.patch
> - compression_parameters
> - replication_parameters (combine strategy + options like we did compression)
> - anything else?

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:

View raw message