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

   Thanks @SEZ9 and @yzeng1618 for the careful current-head follow-up.
   
   I agree that the review discussion should be anchored to the actual current 
head, not the earlier `74ba70f` revision. Since the latest response says the 
source URI sanitization docs/Javadocs and metadata-field scope have changed in 
the current head, I am not going to restate the older findings as Daniel 
blockers in this no-new-commit reply. Please let the current reviewer/author 
discussion converge on the latest head; if a new commit is pushed, I will 
re-review the complete metadata contract again.
   


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