jackylee-ch commented on PR #10162: URL: https://github.com/apache/paimon/pull/10162#issuecomment-5835288224
Superseded by #10166. That PR takes the conservative path for the aggregate engine on the write side — a deduplicate fallback with an explicit warning, and the shared merge-function dispatch raising `NotImplementedError` for the aggregate engine — rather than aggregating in the write buffer. That avoids the `sequence.field` folding-order problem raised in review here (same-key rows would otherwise fold in arrival order, not sequence order). Closing in favour of that approach. Thanks @Akash3121 for the `sequence.field` catch. -- 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]
