Skip to content

fix: parquet statistics#2248

Merged
norberttech merged 1 commit into1.xfrom
parquet-statistics-bug
Mar 5, 2026
Merged

fix: parquet statistics#2248
norberttech merged 1 commit into1.xfrom
parquet-statistics-bug

Conversation

@norberttech
Copy link
Member

Resolves: #2247

Change Log


Added

Fixed

  • dont save string length into min/max statistics

Changed

Removed

Deprecated

Security

dont save string length into min/max statistics
@codecov
Copy link

codecov bot commented Mar 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.02%. Comparing base (899c7e1) to head (1ab923c).
⚠️ Report is 2 commits behind head on 1.x.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##              1.x    #2248      +/-   ##
==========================================
+ Coverage   82.80%   83.02%   +0.21%     
==========================================
  Files        1360     1360              
  Lines       50586    50594       +8     
==========================================
+ Hits        41890    42005     +115     
+ Misses       8696     8589     -107     
Components Coverage Δ
etl 90.17% <ø> (+0.83%) ⬆️
cli 85.76% <ø> (ø)
lib-array-dot 94.82% <ø> (ø)
lib-azure-sdk 60.01% <ø> (ø)
lib-doctrine-dbal-bulk 95.16% <ø> (ø)
lib-filesystem 83.42% <ø> (ø)
lib-types 88.88% <ø> (+1.50%) ⬆️
lib-parquet 68.28% <100.00%> (+0.03%) ⬆️
lib-parquet-viewer 83.04% <ø> (ø)
lib-snappy 90.65% <ø> (+0.93%) ⬆️
lib-dremel 0.00% <ø> (ø)
lib-postgresql 88.15% <ø> (ø)
lib-telemetry 89.57% <ø> (ø)
bridge-filesystem-async-aws 90.95% <ø> (ø)
bridge-filesystem-azure 89.38% <ø> (ø)
bridge-monolog-http 97.45% <ø> (ø)
bridge-openapi-specification 91.50% <ø> (ø)
symfony-http-foundation 74.11% <ø> (ø)
bridge-psr18-telemetry 100.00% <ø> (ø)
bridge-psr7-telemetry 100.00% <ø> (ø)
bridge-telemetry-otlp 89.29% <ø> (+0.25%) ⬆️
bridge-symfony-http-foundation-telemetry 88.23% <ø> (ø)
bridge-symfony-telemetry-bundle 67.60% <ø> (ø)
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 79.29% <ø> (ø)
adapter-text 81.25% <ø> (ø)
adapter-xml 82.07% <ø> (ø)
adapter-avro 0.00% <ø> (ø)
adapter-excel 94.02% <ø> (ø)
adapter-postgresql 0.00% <ø> (ø)
bridge-phpunit-telemetry 48.14% <ø> (ø)
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@norberttech norberttech merged commit fd0c317 into 1.x Mar 5, 2026
23 checks passed
@norberttech norberttech deleted the parquet-statistics-bug branch March 5, 2026 16:00
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.

StatisticsCounter::toStatistics() writes length-prefixed bytes for min_value/max_value on BYTE_ARRAY string columns, causing corrupted statistics

1 participant