Skip to content

Conversation

@dybyte
Copy link
Contributor

@dybyte dybyte commented Jan 2, 2026

Purpose of this pull request

  • Resolve ClassCastException by separating IdentityMapTransform and IdentityFlatMapTransform
  • Fix test using incorrect config file that masked the issue
AbstractMultiCatalogTransform$IdentityTransform cannot be cast to org.apache.seatunnel.api.transform.SeaTunnelFlatMapTransform

Does this PR introduce any user-facing change?

No.

How was this patch tested?

TestSQL.testSQLTransformMultiTable()

Check list

@zhangshenghang
Copy link
Member

Overall, it looks great. Could you add some unit tests?

dybyte added 5 commits January 8, 2026 01:45
…l-transform-multi-table

# Conflicts:
#	seatunnel-transforms-v2/src/test/java/org/apache/seatunnel/transform/sql/SQLMultiCatalogFlatMapTransformTest.java
@dybyte
Copy link
Contributor Author

dybyte commented Jan 7, 2026

@zhangshenghang Added unit tests. PTAL, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants