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

   Thanks @SEZ9. I agree these follow-up points should be resolved or 
explicitly narrowed before merge, especially the widened `isMetadataField` 
semantics and the `SourceUri` / nullability contract details. From Daniel's 
side the PR direction is still coherent, but I would wait for a code update or 
a narrower contract statement before 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