Skip to content

Fix: decimal cast error message - #5821

Merged
connortsui20 merged 1 commit into
developfrom
ct/decimal-cast
Dec 27, 2025
Merged

connortsui20 merged 1 commit into
developfrom
ct/decimal-cast

Conversation

@connortsui20

Copy link
Copy Markdown
Member

This should also make sure that the fuzzer doesn't keep reporting the wrong error message.

As for the TODO, see #5820

@connortsui20 connortsui20 added changelog/fix A bug fix fuzzer Issues detected by the fuzzer labels Dec 26, 2025
@codecov

codecov Bot commented Dec 26, 2025 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.94%. Comparing base (c7b0ef9) to head (1bf72a7).
⚠️ Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
...rtex-array/src/arrays/decimal/compute/fill_null.rs 0.00% 6 Missing ⚠️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@connortsui20
connortsui20 merged commit e2f4f2c into develop Dec 27, 2025
53 checks passed
@connortsui20
connortsui20 deleted the ct/decimal-cast branch December 27, 2025 00:52
amorynan pushed a commit to amorynan/vortex that referenced this pull request Dec 27, 2025
This should also make sure that the fuzzer doesn't keep reporting the
wrong error message.

As for the TODO, see vortex-data#5820

Signed-off-by: Connor Tsui <[email protected]>
Signed-off-by: amorynan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/fix A bug fix fuzzer Issues detected by the fuzzer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants