SEZ9 commented on PR #11746:
URL: https://github.com/apache/seatunnel/pull/11746#issuecomment-6105365485

   Thanks for the follow-up in 34af774c3. As the latest review of this head 
notes, the items from the 832b3b8a5 round are handled on the code side: the 
explicit `supportCompositeKeySplit` opt-outs for `StarRocksDialect`, 
`DsqlDialect`, `OpenGaussDialect` and `PostgresLowDialect` with the 
accompanying `testUnvalidatedDerivedDialectsDisableCompositeKeySplit`, the 
`compareArrays(...) == 0` duplicate-boundary guard, and the EN/ZH 
`incompatible-changes.md` entries plus the `where_condition` limitation 
sentence in `Jdbc.md`. The one remaining open point is the carryover omission 
called out in that review for this head; once it is addressed I'm happy to take 
another pass.
   
   <!-- streview-comment:1675 -->


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to