dramaticlly commented on PR #7751: URL: https://github.com/apache/iceberg/pull/7751#issuecomment-1577053405
> @dramaticlly Thanks for the suggestion. People won't directly use the Python code, nor I would expect a Java/Javascript version of it. It is more for the developer to see the impact on the code when making changes to the yaml. I think having everything in the same folder makes things simpler (otherwise I'd be inclined to add another `README.md` to the `open-api/` dir). I think adding the directory doesn't add much value other than adding more complexity to the directory structure. WDYT? I see, thanks for the explanation. For those generated code, do we want every change in spec to kick off the python generation and include such in PR or do you plan to add some sort of git action to apply it automatically? Anyway, change LGTM -- 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]
