[ 
https://issues.apache.org/jira/browse/CASSANDRA-17623?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Doug Rohrer updated CASSANDRA-17623:
------------------------------------
    Test and Documentation Plan: 
Unit tests at multiple levels added to test.

Github PRs/Circle runs:

 

Trunk: [https://github.com/apache/cassandra/pull/1621]

Circle:

[java8_tests|https://app.circleci.com/pipelines/github/JeetKunDoug/cassandra/18/workflows/d33b2827-b970-4fc2-a56e-4ed68a532005]

[java11_tests|https://app.circleci.com/pipelines/github/JeetKunDoug/cassandra/18/workflows/4631f123-ffde-4151-833c-ecd0b9378f3f]

4.1: [https://github.com/apache/cassandra/pull/1633]

Circle:

[java8_tests|https://app.circleci.com/pipelines/github/JeetKunDoug/cassandra/19/workflows/deb9a03f-53e5-47be-8f6a-a4e60db6e375]

[java11_tests|https://app.circleci.com/pipelines/github/JeetKunDoug/cassandra/19/workflows/ca3763ea-bbb9-42c5-bfc7-3291969f04f0]

4.0: [https://github.com/apache/cassandra/pull/1634]

Circle

[java8_tests|https://app.circleci.com/pipelines/github/JeetKunDoug/cassandra/20/workflows/3a596abf-d173-44a2-b172-e7e74aaa9c3e]

[java11_tests|https://app.circleci.com/pipelines/github/JeetKunDoug/cassandra/20/workflows/a4532032-ef90-42ba-8b13-21b68d0825de]

3.11: [https://github.com/apache/cassandra/pull/1635]

Circle: 
[java8_tests|https://app.circleci.com/pipelines/github/JeetKunDoug/cassandra/22/workflows/eb51d3db-c196-4031-acc6-b36deb060dc7]

3.0: [https://github.com/apache/cassandra/pull/1636]

Circle: 
[java8_tests|https://app.circleci.com/pipelines/github/JeetKunDoug/cassandra/23/workflows/4f6428c2-a4f9-4bee-80d6-22519f0b5c75]

 

Circle runs can be found at 
https://app.circleci.com/pipelines/github/JeetKunDoug/cassandra?filter=all

  was:
Unit tests at multiple levels added to test.

Github PR at https://github.com/apache/cassandra/pull/1621


> Frozen maps may be serialized unsorted, causing inability to query later
> ------------------------------------------------------------------------
>
>                 Key: CASSANDRA-17623
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-17623
>             Project: Cassandra
>          Issue Type: Bug
>          Components: CQL/Semantics
>            Reporter: Doug Rohrer
>            Assignee: Doug Rohrer
>            Priority: Normal
>             Fix For: 4.0.x, 4.1-beta, 4.x
>
>          Time Spent: 2h 40m
>  Remaining Estimate: 0h
>
> CASSANDRA-7859 introduced the ability to use frozen collections as parts of 
> primary keys. This +requires+ all frozen maps to be persisted with their 
> entries sorted by the map keys. If the map is +not+ sorted correctly, it 
> becomes impossible to project all of the map values out of the map using the 
> map projection/selection syntax. For example, the select below would fail if 
> the map was not sorted correctly and the higher-valued key was persisted 
> first:
> {code:sql}
> CREATE TABLE test.test (k text, c frozen<map<text, text>>, PRIMARY KEY (k, 
> c));
> INSERT INTO test.test (k, c) VALUES ('key', {'z':'second_value', 
> 'a':'first_value'});
> SELECT k, c['a'] from test.test where k='key' -- c['a'] would return NULL here
> {code}
> Additionally, if you attempted to select just that row by using the complete 
> map value in a WHERE clause, which is also supported, it would return no rows 
> unless the map provided by the query processor just happened to be sorted the 
> same way as the persisted value.
> However, there is a bug in Maps.java where we don't actually use a SortedMap 
> in {{Maps.Value#fromSerialized}}, which manifests if a client sends an 
> unsorted map as a bound parameter to a query on insert or select. In either 
> case, the map may not be sorted correctly, leading to either invalid data 
> being persisted to disk (in the INSERT case) or the query not being able to 
> be executed/returning 0 rows even though a row _should_ exist (SELECT).
> This bug affects any usage of parameterized queries (tested with the DataStax 
> driver, and was originally discovered when using the CQLSSTableWriter code to 
> write data locally).



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to