Skip to content

fix(sql): fix symbol column initialization in latest by cursors#3027

Merged
ideoma merged 1 commit intomasterfrom
bz_fix_latest_by_symbol_init
Feb 27, 2023
Merged

fix(sql): fix symbol column initialization in latest by cursors#3027
ideoma merged 1 commit intomasterfrom
bz_fix_latest_by_symbol_init

Conversation

@bziobrowski
Copy link
Contributor

Fixes #3026

Fixes symbol filter initialization order in query mentioned in isssue and :

create table a ( i int, s symbol, ts timestamp ) timestamp(ts);

select s, i, ts 
from a 
where s in (select distinct s from a) 
and s = 'ABC' <-- fixed filter 
latest on ts partition by s;

@ideoma
Copy link
Collaborator

ideoma commented Feb 27, 2023

[PR Coverage check]

😍 pass : 2 / 2 (100.00%)

file detail

path covered line new line coverage
🔵 io/questdb/griffin/engine/table/LatestByValuesFilteredRecordCursor.java 1 1 100.00%
🔵 io/questdb/griffin/engine/table/LatestByAllFilteredRecordCursor.java 1 1 100.00%

@ideoma ideoma merged commit d64d337 into master Feb 27, 2023
@ideoma ideoma deleted the bz_fix_latest_by_symbol_init branch February 27, 2023 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

An error occurred after query about LATEST ON <timestamp> PARTITION BY <symbol>

3 participants