Skip to content

refactor(flow-php/etl): remove union columns - #2657

Merged
norberttech merged 1 commit into
1.xfrom
remove-union-definition
Sep 25, 2026
Merged

norberttech merged 1 commit into
1.xfrom
remove-union-definition

Conversation

@norberttech

Copy link
Copy Markdown
Member

Resolves: #xxx

Change Log


Added

Fixed

  • flow-php/etl-adapter-postgresql - array columns are typed list<?T> and can be written to Floe and Parquet

Changed

  • flow-php/etl - null|T inside list, map and structure column types becomes ?T
  • flow-php/etl - any other union inside a column type throws UnsupportedUnionTypeException
  • flow-php/etl-adapter-json - JSON Schema anyOf nested in items or properties is refused

Removed

  • flow-php/etl - UnionDefinition and union_schema()

Deprecated

Security

- drop UnionDefinition, union_schema() and their helpers
- null|T inside list/map/structure types becomes ?T, other unions are refused
- postgresql array columns are list<?T>, writable to Floe and Parquet
- json schema anyOf nested in items/properties is refused
- pin just, md-link-linter image and azurite in CI
@norberttech norberttech added this to the 0.45.0 milestone Sep 25, 2026
@codecov

codecov Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.54%. Comparing base (96edb18) to head (d94cb2a).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff              @@
##                1.x    #2657      +/-   ##
============================================
- Coverage     88.58%   88.54%   -0.04%     
- Complexity        0    26465   +26465     
============================================
  Files          2046     2042       -4     
  Lines         78197    77954     -243     
============================================
- Hits          69272    69028     -244     
- Misses         8925     8926       +1     
Components Coverage Δ
etl 95.30% <100.00%> (-0.09%) ⬇️
cli 87.62% <ø> (ø)
lib-array-dot 96.52% <ø> (ø)
lib-azure-sdk 64.44% <ø> (ø)
lib-doctrine-dbal-bulk 94.24% <ø> (ø)
lib-filesystem 87.61% <ø> (ø)
lib-types 94.42% <ø> (-0.10%) ⬇️
lib-parquet 70.54% <ø> (+0.03%) ⬆️
lib-parquet-viewer 82.26% <ø> (ø)
lib-snappy 89.86% <ø> (ø)
lib-dremel 0.00% <ø> (ø)
lib-postgresql 89.34% <ø> (ø)
lib-telemetry 86.64% <ø> (ø)
bridge-filesystem-async-aws 92.76% <ø> (ø)
bridge-filesystem-azure 90.34% <ø> (ø)
bridge-filesystem-sftp 91.02% <ø> (ø)
bridge-monolog-http 96.84% <ø> (ø)
bridge-monolog-telemetry 94.79% <ø> (ø)
bridge-openapi-specification 91.45% <ø> (ø)
symfony-http-foundation 75.81% <ø> (ø)
bridge-psr18-telemetry 100.00% <ø> (ø)
bridge-psr3-telemetry 98.95% <ø> (ø)
bridge-psr7-telemetry 100.00% <ø> (ø)
bridge-telemetry-otlp 90.11% <ø> (ø)
bridge-symfony-http-foundation-telemetry 92.85% <ø> (ø)
bridge-symfony-filesystem-bundle 91.85% <ø> (ø)
bridge-symfony-filesystem-cache 98.18% <ø> (ø)
bridge-symfony-postgresql-bundle 93.62% <ø> (ø)
bridge-symfony-postgresql-cache 94.41% <ø> (ø)
bridge-symfony-postgresql-messenger 98.80% <ø> (ø)
bridge-symfony-postgresql-session 93.65% <ø> (ø)
bridge-symfony-telemetry-bundle 90.72% <ø> (ø)
adapter-chartjs 80.50% <ø> (ø)
adapter-csv 95.45% <ø> (ø)
adapter-doctrine 93.21% <ø> (ø)
adapter-google-sheet 99.31% <ø> (ø)
adapter-http 88.13% <ø> (ø)
adapter-json 90.97% <ø> (-0.07%) ⬇️
adapter-logger 44.44% <ø> (ø)
adapter-parquet 95.87% <ø> (ø)
adapter-text 95.45% <ø> (ø)
adapter-xml 88.86% <ø> (ø)
adapter-avro 7.14% <ø> (ø)
adapter-excel 95.88% <ø> (ø)
adapter-postgresql 95.79% <100.00%> (ø)
adapter-seal 83.88% <ø> (ø)
bridge-phpunit-postgresql 75.30% <ø> (ø)
bridge-phpunit-telemetry 87.36% <ø> (ø)
bridge-phpstan-types 0.00% <ø> (ø)
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.

@norberttech
norberttech merged commit 7e6bb92 into 1.x Sep 25, 2026
49 of 50 checks passed
@norberttech
norberttech deleted the remove-union-definition branch September 25, 2026 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant