[ 
https://issues.apache.org/jira/browse/CASSANDRA-10162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14712948#comment-14712948
 ] 

Sylvain Lebresne commented on CASSANDRA-10162:
----------------------------------------------

bq. Those could be sorted, but according to an incorrect comparator.

It's a theoretical risk, but I can't see us passing a {{SortedMap}} or 
{{SortedSet}} to those methods without it being the right comparator outside of 
it being a bug in the first place. I'm totally fine removing the checks and 
sorting unconditionally though, I just don't think it's a real problem in 
practice.

> Test regression for  cqlsh_tests.TestCqlsh.test_describe
> --------------------------------------------------------
>
>                 Key: CASSANDRA-10162
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-10162
>             Project: Cassandra
>          Issue Type: Sub-task
>            Reporter: Sylvain Lebresne
>            Assignee: Sylvain Lebresne
>             Fix For: 3.0 beta 2
>
>
> The regression is due to the commit db782362aaf25214a94c393b20cdbade829c5291 
> from CASSANDRA-6717, the one that switched to using frozen sets and maps in 
> schema tables.
> The problem is that after this patch, the maps and sets are not sorted 
> anymore, which confuse the test in question. Now, we document that maps and 
> sets are sorted, and that should be true for frozen ones too so it's a legit 
> bug imo, not a problem of the test.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to