Skip to content

Conversation

@connortsui20
Copy link
Contributor

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

As for the TODO, see #5820

Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>
@connortsui20 connortsui20 added changelog/fix A bug fix fuzzer Issues detected by the fuzzer labels Dec 26, 2025
@codecov
Copy link

codecov bot commented Dec 26, 2025

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
54 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 <connor.tsui20@gmail.com>
Signed-off-by: amorynan <amorywang111@gmail.com>
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.

3 participants