[
https://issues.apache.org/jira/browse/CASSANDRA-7859?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14200421#comment-14200421
]
Sylvain Lebresne commented on CASSANDRA-7859:
---------------------------------------------
bq. When frozen lists and maps are used as cell values, the collection value
ordering doesn't matter.
I'm confused because my earlier comment was exactly to say that I'm
uncomfortable making that assumption and you seemed to agree initially. To
reiter, collection values are sorted (even when used as cell values), and so it
felt fishy to me to consider that changing to a type that change the internal
sorting of those values is "compatible". Typically sending a value to clients
that is not properly sorted feels wrong. And given we lost little in practice
by requiring the sorting not to change, I'd prefer sticking to that for now.
bq. I think it could potentially introduce assertion errors when the behavior
is reasonable.
Fair enough, it's just that the comments in there pretty much says that it
should only be call for frozen collections. Maybe we should just remove the
comment in that case. Not a big deal though.
> Extend freezing to collections
> ------------------------------
>
> Key: CASSANDRA-7859
> URL: https://issues.apache.org/jira/browse/CASSANDRA-7859
> Project: Cassandra
> Issue Type: Improvement
> Reporter: Sylvain Lebresne
> Assignee: Tyler Hobbs
> Labels: cql
> Fix For: 2.1.2
>
> Attachments: 7859-v1.txt
>
>
> This is the follow-up to CASSANDRA-7857, to extend {{frozen}} to collections.
> This will allow things like {{map<text, frozen<map<int, int>>>}} for
> instance, as well as allowing {{frozen}} collections in PK columns.
> Additionally (and that's alsmot a separate ticket but I figured we can start
> discussing it here), we could decide that tuple is a frozen type by default.
> This means that we would allow {{tuple<int, text>}} without needing to add
> {{frozen}}, but we would require {{frozen}} for complex type inside tuple, so
> {{tuple<int, list<text>>}} would be rejected, but not {{tuple<int,
> frozen<list<text>>>}}.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
