Skip to content

IGNITE-28896 SQL Calcite: Support BigDecimal for FETCH, OFFSET and LIMIT#13375

Open
tkalkirill wants to merge 4 commits into
masterfrom
ignite-28896
Open

IGNITE-28896 SQL Calcite: Support BigDecimal for FETCH, OFFSET and LIMIT#13375
tkalkirill wants to merge 4 commits into
masterfrom
ignite-28896

Conversation

@tkalkirill

Copy link
Copy Markdown
Contributor

@tkalkirill tkalkirill changed the title IGNITE-28896 Wip IGNITE-28896 SQL Calcite: Support BigDecimal for FETCH, OFFSET and LIMIT Jul 15, 2026
@zstan

zstan commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

So, it need to be merged somehow ? #13311
I don`t understand about rounding policy, this policy need to be applied globally for all rounding operations not only for limit\offset

@tkalkirill

tkalkirill commented Jul 20, 2026

Copy link
Copy Markdown
Contributor Author

@zstan BigDecimal is already in Calcite's main branch, so I think it makes sense to use that.
Regarding the rounding policy: the name FetchOffsetRoundingPolicy implies it doesn't cover all rounding, but specifically OFFSET and FETCH, and the documentation describes it that way as well.

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