Skip to content

Handle runtime Parquet Hadoop API version via reflection#330

Merged
EnricoMi merged 1 commit intomasterfrom
parquet-api-reflection
Dec 11, 2025
Merged

Handle runtime Parquet Hadoop API version via reflection#330
EnricoMi merged 1 commit intomasterfrom
parquet-api-reflection

Conversation

@EnricoMi
Copy link
Contributor

@EnricoMi EnricoMi commented Dec 10, 2025

Runtime and compile time version of Parquet Hadoop and its dependencies can vary. The runtime version is either given by Spark or by the user. This replaces Spark version specific code with single implementation for all Spark and Parquet Hadoop versions. Compile time uses the latest Parquet Hadoop version, runtime checks existence of methods in actual version via reflection.

@github-actions
Copy link

github-actions bot commented Dec 10, 2025

Test Results

   615 files  ±0     615 suites  ±0   1h 32m 3s ⏱️ -30s
   530 tests ±0     528 ✅ ±0    2 💤 ±0  0 ❌ ±0 
21 343 runs  ±0  20 773 ✅ ±0  570 💤 ±0  0 ❌ ±0 

Results for commit d18c7af. ± Comparison against base commit b383b49.

♻️ This comment has been updated with latest results.

@EnricoMi EnricoMi force-pushed the parquet-api-reflection branch 2 times, most recently from 55dd08a to 5629587 Compare December 10, 2025 10:22
EnricoMi added a commit that referenced this pull request Dec 10, 2025
Running tests against the packaged jar helps identifying issues with dependencies and shaded classes.
- Runs unit tests through Maven surefire plugin, Scala and Java tests together.
- Runs integration tests through Maven failsafe plugin, Scala and Java tests together.

Needed by #329, #330.
EnricoMi added a commit that referenced this pull request Dec 10, 2025
EnricoMi added a commit that referenced this pull request Dec 10, 2025
@EnricoMi EnricoMi force-pushed the parquet-api-reflection branch 2 times, most recently from d660b45 to 36083e1 Compare December 11, 2025 06:54
@EnricoMi EnricoMi force-pushed the parquet-api-reflection branch from 36083e1 to d18c7af Compare December 11, 2025 11:30
@EnricoMi EnricoMi merged commit 0a446f2 into master Dec 11, 2025
144 of 155 checks passed
@EnricoMi EnricoMi deleted the parquet-api-reflection branch December 11, 2025 18:53
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.

1 participant