WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
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

@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 <[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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants