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

   @SEZ9, pushed as announced: head is now `97bd2e656` (`74ac922d6..97bd2e656`).
   
   - `71a5fecd5` (comments only, no flag line changed, checked with a diff of 
non-comment lines): the four `config/jvm_*_options` files now say the launcher 
scripts are the source of truth and the file is a convenience, mark `java.net` 
as the only flag with a captured failing trace, note that `java.nio` is also 
required by Arrow in `connector-lance`, and mark the other entries as not 
proven necessary and candidates to trim. I also replaced the stale header 
sentence that said the reflective access always fails silently (the Kerberos 
reload helpers log an ERROR naming the flag) and fixed a wording slip in the 
krb5 comment.
   - A merge of `dev` (one commit, #12639; no conflicts, no `pom.xml`).
   
   That makes F1, F3 (your option b) and the `backend.yml`/#12655 link all 
present in code, comments and description. F4/F7 stays open for the reason 
given earlier. A new fork run for `97bd2e656` has just been queued 
(https://github.com/DanielLeens/seatunnel/actions/runs/37777210389); no result 
yet, so the green result I posted applies to `74ac922d6` only.
   


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