Skip to content

[Variant] Enhance the variant fuz test to cover time/timestamp/uuid primitive type#8200

Merged
alamb merged 1 commit intoapache:mainfrom
klion26:improve_fuzzy_test
Aug 21, 2025
Merged

[Variant] Enhance the variant fuz test to cover time/timestamp/uuid primitive type#8200
alamb merged 1 commit intoapache:mainfrom
klion26:improve_fuzzy_test

Conversation

@klion26
Copy link
Copy Markdown
Member

@klion26 klion26 commented Aug 21, 2025

Which issue does this PR close?

Rationale for this change

Add logic for the fuzzy test to cover time/timestampnanos/uuid.

What changes are included in this PR?

  • Add more cases in Variant Fuzzy Testing to cover the time/TimestampNanos/UUID

Are these changes tested?

Covered by the existing test

Are there any user-facing changes?

No

@github-actions github-actions bot added the parquet-variant parquet-variant* crates label Aug 21, 2025
@klion26
Copy link
Copy Markdown
Member Author

klion26 commented Aug 21, 2025

@carpecodeum @alamb Please help to review this when you're free, thanks.

@alamb alamb changed the title Enhance the variant fuzzy test to cover time/timestamp/uuid primitive type [Variant] Enhance the variant fuzzy test to cover time/timestamp/uuid primitive type Aug 21, 2025
Copy link
Copy Markdown
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

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

Thank you @klion26 -- this looks good to me

@alamb alamb changed the title [Variant] Enhance the variant fuzzy test to cover time/timestamp/uuid primitive type [Variant] Enhance the variant fuz test to cover time/timestamp/uuid primitive type Aug 21, 2025
@alamb alamb merged commit d5701d2 into apache:main Aug 21, 2025
14 checks passed
@klion26 klion26 deleted the improve_fuzzy_test branch August 22, 2025 02:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

parquet-variant parquet-variant* crates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Variant] Improve fuzz test for Variant

2 participants