Skip to content

fix(java): support skip optional sql serializers for java11+#3553

Merged
chaokunyang merged 8 commits intoapache:mainfrom
chaokunyang:issue-1695-lazy-class-load
Apr 12, 2026
Merged

fix(java): support skip optional sql serializers for java11+#3553
chaokunyang merged 8 commits intoapache:mainfrom
chaokunyang:issue-1695-lazy-class-load

Conversation

@chaokunyang
Copy link
Copy Markdown
Collaborator

@chaokunyang chaokunyang commented Apr 10, 2026

Why?

What does this PR do?

  • move optional SQL serializer access behind reflective helpers so startup does not touch java.sql unless those classes are actually used
  • add a JPMS regression test for building without java.sql and preserve time ref-tracking behavior for lazy serializer registrations

Related issues

Fixes #1695

AI Contribution Checklist

  • Substantial AI assistance was used in this PR: yes / no
  • If yes, I included a completed AI Contribution Checklist in this PR description and the required AI Usage Disclosure.
  • If yes, my PR description includes the required ai_review summary and screenshot evidence of the final clean AI review results from both fresh reviewers on the current PR diff or current HEAD after the latest code changes.

Does this PR introduce any user-facing change?

  • Does this PR introduce any public API change?
  • Does this PR introduce any binary protocol compatibility change?

@chaokunyang chaokunyang changed the title fix(java): lazily resolve optional sql serializers fix(java): support skip optional sql serializers for java11+ Apr 12, 2026
@chaokunyang chaokunyang requested a review from theweipeng April 12, 2026 02:26
@chaokunyang chaokunyang merged commit 1d8db73 into apache:main Apr 12, 2026
61 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fury eagerly loads non-base module classes

2 participants