Skip to content

[SLT] Fixes for SLT failing tests#5852

Merged
snkas merged 2 commits intomainfrom
slt
Mar 18, 2026
Merged

[SLT] Fixes for SLT failing tests#5852
snkas merged 2 commits intomainfrom
slt

Conversation

@mihaibudiu
Copy link
Contributor

  • Skip two tests that overflow in integer multiplication; this result is produced by Postgres as well
  • Optimize essentially CAST(NULL AS BOOLEAN) to FALSE. This makes another failing test pass, since it no longer performs a division by zero, since the result is optimized away to NULL. Note that this behavior is allowed by the SQL standard.

Describe Manual Test Plan

Ran all the tests that failed in CI manually.

Signed-off-by: Mihai Budiu <mbudiu@feldera.com>
Signed-off-by: Mihai Budiu <mbudiu@feldera.com>
@mihaibudiu mihaibudiu requested a review from snkas March 18, 2026 00:16
@mihaibudiu mihaibudiu enabled auto-merge March 18, 2026 00:16
Copy link

@mythical-fred mythical-fred left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mihaibudiu mihaibudiu added this pull request to the merge queue Mar 18, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 18, 2026
@snkas snkas added this pull request to the merge queue Mar 18, 2026
Merged via the queue into main with commit 2f2401d Mar 18, 2026
1 check passed
@snkas snkas deleted the slt branch March 18, 2026 11:21
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.

3 participants