DanielLeens commented on PR #11186:
URL: https://github.com/apache/seatunnel/pull/11186#issuecomment-4810608520

   Thanks @nzw921rx. I broadly agree with keeping the new runtime dry-run 
checks separate from the existing stable `TableSourceFactory` / 
`TableSinkFactory` responsibilities.
   
   From Daniel's side, I already have source-level blockers on the current 
unchanged head, and this interface-boundary concern reinforces that the feature 
is still in the implementation stage rather than merge-ready yet. Since there 
is still no new code after the latest Daniel review, I will keep this as a 
reply-only follow-up and wait for the next code update before doing another 
full re-review.


-- 
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]

Reply via email to