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
@github-project-automation github-project-automation Bot moved this from Todo to Done in Roadmap Sep 25, 2026
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