[jira] [Commented] (CASSANDRA-19461) SAI does not index empty bytes even for types that allow empty bytes as a valid input
[ https://issues.apache.org/jira/browse/CASSANDRA-19461?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17829558#comment-17829558 ] Ekaterina Dimitrova commented on CASSANDRA-19461: - [~dcapwell] already reviewed it so I will move away :) > SAI does not index empty bytes even for types that allow empty bytes as a > valid input > - > > Key: CASSANDRA-19461 > URL: https://issues.apache.org/jira/browse/CASSANDRA-19461 > Project: Cassandra > Issue Type: Bug > Components: Feature/SAI >Reporter: Caleb Rackliffe >Assignee: Caleb Rackliffe >Priority: Normal > Fix For: 5.0-rc, 5.x > > Attachments: ci_summary.html, result_details.tar.gz > > Time Spent: 1h 20m > Remaining Estimate: 0h > > This is easy to reproduce with a test that looks something like this: > {noformat} > @Test > public void testEmptyString() > { > createTable("CREATE TABLE %s (k TEXT PRIMARY KEY, v text)"); > createIndex(String.format(CREATE_INDEX_TEMPLATE, 'v')); > execute("INSERT INTO %s (k, v) VALUES ('0', '')"); > execute("INSERT INTO %s (k) VALUES ('1')"); > > // flush(); < there is not always a memtable index involved, a fix > will have to pay attention to this > List rows = executeNet("SELECT * FROM %s WHERE v = ''").all(); > assertEquals(1, rows.size()); <— FAILS! No matches... > } > {noformat} -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-19461) SAI does not index empty bytes even for types that allow empty bytes as a valid input
[ https://issues.apache.org/jira/browse/CASSANDRA-19461?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17829337#comment-17829337 ] Ekaterina Dimitrova commented on CASSANDRA-19461: - I'll review it tomorrow when we have CI also ready. Thanks > SAI does not index empty bytes even for types that allow empty bytes as a > valid input > - > > Key: CASSANDRA-19461 > URL: https://issues.apache.org/jira/browse/CASSANDRA-19461 > Project: Cassandra > Issue Type: Bug > Components: Feature/SAI >Reporter: Caleb Rackliffe >Assignee: Caleb Rackliffe >Priority: Normal > Fix For: 5.0-rc, 5.x > > Time Spent: 1h 10m > Remaining Estimate: 0h > > This is easy to reproduce with a test that looks something like this: > {noformat} > @Test > public void testEmptyString() > { > createTable("CREATE TABLE %s (k TEXT PRIMARY KEY, v text)"); > createIndex(String.format(CREATE_INDEX_TEMPLATE, 'v')); > execute("INSERT INTO %s (k, v) VALUES ('0', '')"); > execute("INSERT INTO %s (k) VALUES ('1')"); > > // flush(); < there is not always a memtable index involved, a fix > will have to pay attention to this > List rows = executeNet("SELECT * FROM %s WHERE v = ''").all(); > assertEquals(1, rows.size()); <— FAILS! No matches... > } > {noformat} -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org