Skip to content

Don't allow scanning tables with defined output ordering#2429

Merged
AdamGS merged 2 commits intodevelopfrom
adamg/verify-no-output-ordering
Feb 19, 2025
Merged

Don't allow scanning tables with defined output ordering#2429
AdamGS merged 2 commits intodevelopfrom
adamg/verify-no-output-ordering

Conversation

@AdamGS
Copy link
Contributor

@AdamGS AdamGS commented Feb 19, 2025

Error when trying to query tables defined with a WITH ORDER clause, e.g.:

CREATE EXTERNAL TABLE tbl (
    c1  VARCHAR NOT NULL,
    c2  INT NOT NULL,
)
STORED AS VORTEX
WITH ORDER (c1 ASC)

Until now we effectively just ignored these options, but that might have some correctness issues so I figured its just best to error out right now.

@AdamGS AdamGS mentioned this pull request Feb 8, 2025
12 tasks
@AdamGS AdamGS enabled auto-merge (squash) February 19, 2025 15:44
@AdamGS AdamGS merged commit 516f941 into develop Feb 19, 2025
18 checks passed
@AdamGS AdamGS deleted the adamg/verify-no-output-ordering branch February 19, 2025 17:08
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.

2 participants