Skip to content

fix[delta]: compress with nulls undef values use last valid value (or 0)#7124

Merged
joseph-isaacs merged 1 commit intodevelopfrom
ji/fix-fuzzer-7108
Mar 23, 2026
Merged

fix[delta]: compress with nulls undef values use last valid value (or 0)#7124
joseph-isaacs merged 1 commit intodevelopfrom
ji/fix-fuzzer-7108

Conversation

@joseph-isaacs
Copy link
Contributor

When we have undef values this can affect decompression with a undef value (use the previous valid value instead).

Signed-off-by: Joe Isaacs <joe.isaacs@live.co.uk>
@joseph-isaacs joseph-isaacs requested a review from 0ax1 March 23, 2026 11:42
@joseph-isaacs joseph-isaacs marked this pull request as ready for review March 23, 2026 11:42
@joseph-isaacs joseph-isaacs enabled auto-merge (squash) March 23, 2026 11:42
@joseph-isaacs joseph-isaacs added the changelog/fix A bug fix label Mar 23, 2026
use vortex_array::IntoArray;
use vortex_array::ToCanonical;

use crate::bitpack_compress::bitpack_encode;
Copy link
Contributor

Choose a reason for hiding this comment

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

use crate::bitpack_compress::bitpack_encode;
use crate::delta_compress;

Move up?

@joseph-isaacs joseph-isaacs merged commit 1c8667c into develop Mar 23, 2026
58 of 59 checks passed
@joseph-isaacs joseph-isaacs deleted the ji/fix-fuzzer-7108 branch March 23, 2026 12:07
@robert3005
Copy link
Contributor

Thanks @joseph-isaacs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/fix A bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants