stevenschlansker commented on PR #3714: URL: https://github.com/apache/fory/pull/3714#issuecomment-4847883472
@chaokunyang , this is not 100% done yet, but I think it is getting very close. Would you mind doing a design review to make sure that this feature will be acceptable to commit into the project? It is not a small feature, but I think the complexity is justified by giving the row format a low-cost method of achieving schema evolution. If there is some way to streamline the eventual review of a large volume of LLM-typed code, please let me know. I have tried to keep the bar for quality high (many, many review iterations per ai policy) and testing thorough. I considered breaking it up into multiple PRs but there is not a "slice" of the feature that stands on its own. I intend to test this over the coming month or so internally, and mark the PR as ready once we have some real world experience running it. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
