pan3793 opened a new pull request, #64:
URL: https://github.com/apache/kyuubi-shaded/pull/64

   <!--
   Thanks for sending a pull request!
   
   Here are some tips for you:
     1. If this is your first time, please read our contributor guidelines: 
https://kyuubi.readthedocs.io/en/latest/community/CONTRIBUTING.html
     2. If the PR is related to an issue in 
https://github.com/apache/kyuubi/issues, add '[KYUUBI-SHADED #XXXX]' in your PR 
title, e.g., '[KYUUBI-SHADED #XXXX] Your PR title ...'.
     3. If the PR is unfinished, add '[WIP]' in your PR title, e.g., 
'[WIP][KYUUBI-SHADED #XXXX] Your PR title ...'.
   -->
   
   ### _Why are the changes needed?_
   <!--
   Please clarify why the changes are needed. For instance,
     1. If you add a feature, you can talk about the use case of it.
     2. If you fix a bug, you can clarify why it is a bug.
   -->
   To allow Kyuubi to compile with `-release:8` in JDK 9+, we must cut out 
direct access `sun.misc.Signal`, this PR achieves it by creating wrapper 
classes of `sun.misc.Signal` and `sun.misc.SignalHandler` in the Kyuubi Shaded 
project.
   
   Currently, both Kyuubi and Kyuubi Shaded use Java 8 for release. After this 
change, Kyuubi can use any of Java 8+ to compile with `-release:8` while still 
ensuring compatibility with Java 8.
   
   ### _How was this patch tested?_
   
   Have done a PoC - modify Kyuubi code to use the wrapper Signal classes, and 
compile with JDK 21, run on JDK 8, everything works as expected.
   


-- 
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: notifications-unsubscr...@kyuubi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@kyuubi.apache.org
For additional commands, e-mail: notifications-h...@kyuubi.apache.org

Reply via email to