You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@kyuubi.apache.org by "wangzhigang1999 (via GitHub)" <gi...@apache.org> on 2026/09/28 06:44:51 UTC
[PR] [COMMON][JDBC] Validate class types before reflective instantiation [kyuubi]
wangzhigang1999 opened a new pull request, #7769:
URL: https://github.com/apache/kyuubi/pull/7769
### Why are the changes needed?
Several reflective loading paths can initialize or construct a class before checking its expected type.
Use a shared type check before initialization or construction for exception reconstruction and provider, plugin, strategy, and JDBC driver loading. Keep the existing constructor selection and exception reconstruction for supported classes.
### How was this patch tested?
- 23 targeted tests passed across common, JDBC client, server, and Data Agent modules, including checks that rejected classes are not initialized or constructed.
- Manual checks confirmed missing-dependency error wrapping for session advisors, group providers, and Data Agent providers.
- Spotless and `git diff --check` passed.
### Was this patch assisted by generative AI tooling?
Assisted-by: Codex (GPT-6)
--
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-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org
Re: [PR] [COMMON][JDBC] Validate class types before reflective instantiation [kyuubi]
Posted by "aajisaka (via GitHub)" <gi...@apache.org>.
aajisaka closed pull request #7769: [COMMON][JDBC] Validate class types before reflective instantiation
URL: https://github.com/apache/kyuubi/pull/7769
--
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-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org
Re: [PR] [COMMON][JDBC] Validate class types before reflective instantiation [kyuubi]
Posted by "aajisaka (via GitHub)" <gi...@apache.org>.
aajisaka commented on PR #7769:
URL: https://github.com/apache/kyuubi/pull/7769#issuecomment-5966710788
**Merge Summary:**
- merged into master https://github.com/apache/kyuubi/commit/2bf326eaf9bda3402a325cb8b9589a53c124b8e5
- merged into branch-1.12 https://github.com/apache/kyuubi/commit/f76bb9e5f19ee88bfb963974a43aa1f601ef2de7
*Posted by `merge_kyuubi_pr.py`*
--
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-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org