Skip to content

Correction of handling of null counts metadata in parquet files for PhpParquetEngine#2353

Merged
norberttech merged 4 commits into
flow-php:1.xfrom
gCass:fix/parquet_correct_null_metadata
May 8, 2026
Merged

Correction of handling of null counts metadata in parquet files for PhpParquetEngine#2353
norberttech merged 4 commits into
flow-php:1.xfrom
gCass:fix/parquet_correct_null_metadata

Conversation

@gCass
Copy link
Copy Markdown
Contributor

@gCass gCass commented May 8, 2026

Resolves: #2349

Change Log


Added

Fixed

  • Correction of handling of null counts metadata in parquet files for PhpParquetEngine

Changed

Removed

Deprecated

Security

@gCass gCass requested a review from norberttech as a code owner May 8, 2026 08:31
@codecov
Copy link
Copy Markdown

codecov Bot commented May 8, 2026

Codecov Report

❌ Patch coverage is 88.23529% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.04%. Comparing base (4b4cf7d) to head (2e5907b).
⚠️ Report is 9 commits behind head on 1.x.

Additional details and impacted files
@@            Coverage Diff            @@
##                1.x    #2353   +/-   ##
=========================================
  Coverage     85.03%   85.04%           
- Complexity    19989    19994    +5     
=========================================
  Files          1557     1557           
  Lines         58969    58986   +17     
=========================================
+ Hits          50147    50162   +15     
- Misses         8822     8824    +2     
Components Coverage Δ
etl 90.63% <ø> (ø)
cli 85.76% <ø> (ø)
lib-array-dot 94.82% <ø> (ø)
lib-azure-sdk 61.13% <ø> (ø)
lib-doctrine-dbal-bulk 95.16% <ø> (ø)
lib-filesystem 86.51% <ø> (ø)
lib-types 89.38% <ø> (ø)
lib-parquet 68.86% <88.23%> (+0.03%) ⬆️
lib-parquet-viewer 83.04% <ø> (ø)
lib-snappy 90.65% <ø> (ø)
lib-dremel 0.00% <ø> (ø)
lib-postgresql 88.98% <100.00%> (ø)
lib-telemetry 90.32% <ø> (ø)
bridge-filesystem-async-aws 92.63% <ø> (ø)
bridge-filesystem-azure 91.48% <ø> (ø)
bridge-monolog-http 98.70% <ø> (ø)
bridge-monolog-telemetry 90.32% <ø> (ø)
bridge-openapi-specification 91.50% <ø> (ø)
symfony-http-foundation 79.45% <ø> (ø)
bridge-psr18-telemetry 100.00% <ø> (ø)
bridge-psr3-telemetry 97.61% <ø> (ø)
bridge-psr7-telemetry 100.00% <ø> (ø)
bridge-telemetry-otlp 89.64% <ø> (ø)
bridge-symfony-http-foundation-telemetry 88.23% <ø> (ø)
bridge-symfony-filesystem-bundle 94.08% <ø> (ø)
bridge-symfony-filesystem-cache 97.93% <ø> (ø)
bridge-symfony-postgresql-bundle 94.01% <ø> (ø)
bridge-symfony-postgresql-cache 93.52% <ø> (ø)
bridge-symfony-postgresql-messenger 99.59% <ø> (ø)
bridge-symfony-postgresql-session 93.46% <ø> (ø)
bridge-symfony-telemetry-bundle 68.11% <ø> (ø)
adapter-chartjs 84.72% <ø> (ø)
adapter-csv 88.65% <ø> (ø)
adapter-doctrine 92.00% <ø> (ø)
adapter-elasticsearch 92.78% <ø> (ø)
adapter-google-sheet 99.02% <ø> (ø)
adapter-http 67.91% <ø> (ø)
adapter-json 87.61% <ø> (ø)
adapter-logger 47.36% <ø> (ø)
adapter-parquet 82.78% <ø> (ø)
adapter-text 81.25% <ø> (ø)
adapter-xml 82.61% <ø> (ø)
adapter-avro 0.00% <ø> (ø)
adapter-excel 94.02% <ø> (ø)
adapter-postgresql 90.57% <ø> (ø)
bridge-phpunit-postgresql 72.97% <ø> (ø)
bridge-phpunit-telemetry 60.33% <ø> (ø)
bridge-postgresql-valinor 100.00% <ø> (ø)
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@gCass gCass changed the title fixed parquet metadata error Correction of handling of null counts metadata in parquet files for PhpParquetEngine May 8, 2026
@norberttech norberttech merged commit 0a47bb2 into flow-php:1.x May 8, 2026
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Parquet null metadata non correctly evaluated

3 participants