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

   Thanks @roohom, that is completely understandable, and thank you for the 
earlier offer and for the original groundwork in #10134. Nobody needs to wait 
on you for the live run.
   
   To keep expectations clear, the live OSS check (Issue 4 in my last review) 
is still an open item on this PR rather than something your reply resolves: the 
current tests only prove that the `fs.oss.*` options reach the catalog option 
map. A few ways to close it: someone with OSS access (the PR author or a 
maintainer) runs a one-time open/create-table/write/commit/readback against the 
documented provider and posts the result here, or a committer explicitly 
decides this can merge on the option-forwarding scope alone with the limitation 
stated in the docs. The DEBUG credential-logging point (Issue 1) and the doc 
corrections (Issues 2 and 3) are independent of OSS access and can be handled 
in the code and docs directly.
   
   Thanks again for supporting this work.


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