Skip to content

test(data): introduce avro orc and parquet fixtures#139

Open
zjw1111 wants to merge 1 commit into
apache:mainfrom
zjw1111:migrate/test-data-formats
Open

test(data): introduce avro orc and parquet fixtures#139
zjw1111 wants to merge 1 commit into
apache:mainfrom
zjw1111:migrate/test-data-formats

Conversation

@zjw1111

@zjw1111 zjw1111 commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Purpose

Linked issue: N/A

Introduce migrated Avro, ORC, and Parquet test data fixtures under test/test_data/.

This PR keeps the original fixture paths from the source repository and includes:

  • test/test_data/avro
  • test/test_data/orc
  • test/test_data/parquet

README files were normalized for trailing whitespace and final newlines so the staged patch passes whitespace checks. No additional dependency files are included.

No LICENSE or NOTICE updates are required because the source repository declarations do not contain entries for these fixture paths. External contributor analysis did not hit the batch-level Co-authored-by threshold, so no trailer was added.

Tests

API and Format

No API changes. This adds test fixtures only and does not change storage format definitions or protocols.

Documentation

No documentation changes.

Generative AI tooling

Migrated-by: OpenAI Codex

Copilot AI review requested due to automatic review settings July 1, 2026 03:42
@zjw1111

zjw1111 commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

Thanks @fafacao86, @SteNicholas for your previous contributions to paimon-cpp. This migration PR carries that work forward into Apache Paimon C++.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@lxy-9602

lxy-9602 commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

+1

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants