You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@paimon.apache.org by "thswlsqls (via GitHub)" <gi...@apache.org> on 2026/07/08 21:29:08 UTC
[PR] [api] Recurse into element type in MultisetType field-id comparison [paimon]
thswlsqls opened a new pull request, #8519:
URL: https://github.com/apache/paimon/pull/8519
### Purpose
fix #8516
- `MultisetType` did not override `equalsIgnoreFieldId(DataType)` or `isPrunedFrom(Object)`, so it inherited the `DataType` base methods that just `return equals(o)` and compare element field ids.
- For `MULTISET<ROW<...>>` this made ignore-field-id comparison and pruning wrongly return false, corrupting schema-evolution comparison up the chain (`DataField.equalsIgnoreFieldId` / `RowType.equalsIgnoreFieldId`).
- Add both overrides recursing into `elementType`, matching sibling `ArrayType`/`MapType`/`RowType`/`VectorType`.
### Tests
- Added `MultisetTypeTest` covering positive (differing field ids equal, pruned element) and negative (different name/type) cases for both methods.
- `mvn -pl paimon-api test` — MultisetTypeTest 4 tests passed; module `clean install` green (no profiles).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [api] Recurse into element type in MultisetType field-id comparison [paimon]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi merged PR #8519:
URL: https://github.com/apache/paimon/pull/8519
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org