xushiyan commented on pull request #4201: URL: https://github.com/apache/hudi/pull/4201#issuecomment-985305969
> @YannByron Why revert it? I've added a fix for [HUDI-2909](https://issues.apache.org/jira/browse/HUDI-2909) : #4203 Can you please review? the key consideration is: keygen related change should ensure backward-compatible for existing user tables. The previous PR https://github.com/apache/hudi/pull/3944 solves inconsistent behavior but may break for cases like reported in HUDI-2909. The revert was meant to keep things unchanged for users. Inconsistency comes from enabling row-writer, which can be mitigated by disabling row-writer. Looks like #4203 makes a further fix. is there any compatibility concern if rolled out ? If so, would it be easy for user to migrate and adapt to new keygen's behavior? and how to clarify it in the release notes. Just some points to consider. @codope @leesf @nsivabalan -- 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]
