cassandra-loader icon indicating copy to clipboard operation
cassandra-loader copied to clipboard

treat schema parameter as case sensitive

Open al3xandru opened this issue 9 years ago • 6 comments
trafficstars

Do all the necessary escaping internally so schema parameter can be treated as case sensitive

al3xandru avatar Mar 05 '16 23:03 al3xandru

The current version does allow you to use case-sensitive keyspace, table, and column names. For example, you can do -schema 'myks."MyTable"(col1, "Col2")' Does that cover the use case you are looking to address?

brianmhess avatar May 10 '16 23:05 brianmhess

That gets unnecessarily cumbersome very fast.

al3xandru avatar May 10 '16 23:05 al3xandru

The double quotes standard around upper case keyspaces and columns is a cassandra standard. You have to do it in cqlsh, at the driver, etc.

I see your point about it being a bit cumbersome however and am having trouble thinking of downsides of the change. What happens if there's a table called myTable and another table called mytable. Would this be able to tell the difference?

phact avatar Jul 01 '16 20:07 phact

Based on this https://docs.datastax.com/en/cql/3.3/cql/cql_reference/ucase-lcase_r.html I think requiring double quotes to get case sensitivity is necessary (even if a bit cumbersome).

brianmhess avatar Jul 02 '16 12:07 brianmhess

With JSON, this becomes a bigger issue because now we're requiring the column names to come escaped in the JSON file. Nobody will have this. I think it's time to re-evaluate

phact avatar Aug 17 '16 21:08 phact

Here's a new PR to deal with the JSON issue.

phact avatar Aug 17 '16 22:08 phact