SEZ9 commented on PR #11512: URL: https://github.com/apache/seatunnel/pull/11512#issuecomment-6393548886
Thanks for the update. My review of `9170d5e78af33f4e6d0291576ad2840585013f8d` already covers the conflict resolution in `5e346c6844` and the F7 constructor Javadoc in `9170d5e78a`, so I won't repeat that here. One open question: in my earlier follow-up I noted I had not treated the documentation edit as already committed. Between `d2f8193c97` and `9170d5e78a` the net diff only touches the F7 Javadoc, an import-only test merge, and the serializer-ID renumbering, so I don't see a documentation change on this head. Is that edit still planned as a separate push? If so, please let me know when it lands and I will re-check the updated diff. The serializer bounds observation remains a non-blocking follow-up. <!-- streview-comment:2025 --> -- 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]
