SEZ9 commented on issue #12928: URL: https://github.com/apache/seatunnel/issues/12928#issuecomment-6355844268
@Nikk8091 Thanks for picking this up and for the update — the scope you described matches what I had in mind, and reusing the #12443 helper while marking it as a dependency rather than adding a second conversion policy is the right call. To close the question from your first comment: preparing the regressions on dev is fine; the change just needs #12443 merged before it can be finalized. Once #12443 lands and you've rebased and re-verified on dev, please drop a note here and I'll take a look at the review. <!-- streview-comment:1960 --> -- 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]
