-
Notifications
You must be signed in to change notification settings - Fork 227
8347826: Introspector shows wrong method list after 8071693 #3618
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
👋 Welcome back rmarchenko! A progress list of the required criteria for merging this PR into |
❗ This change is not yet ready to be integrated. |
This backport pull request has now been updated with issue from the original commit. |
|
/approval request I'd like to backport this to 17u. The backport is almost clean. PR checks are OK. This backport fixes the wrong method and property list returning by Introspector broken after 8071693 in 17.0.14. |
Hi, this request is about backporting fix to Java 17 (JDK-8347826). Backport request for same fix on Java 21 was completed 2 weeks back (JDK-8358779). If someone can take a look, it would be very helpful. |
@GoeLin , Sir, you validated same request for JDK21 last month. Can you please help to move this PR as well. |
@wkia, PR still pending approval. We are eagerly waiting for it to merge in main branch so that we are able to upgrade to latest stable JDK17. |
@anuj-ion AFAIU the change was deferred to the next release, at least the JDK bug's label seems so. |
@wkia , Ah, I didn't notice the label. Thanks for sharing this info. Will it be a candidate for 17.0.18? |
@anuj-ion yes, I hope so. |
This is backport of "8347826: Introspector shows wrong method list after 8071693"
The backport is almost clean, the only conflicts were in copyright year in 2 files.
Progress
Issue
Reviewing
Using
git
Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk17u-dev.git pull/3618/head:pull/3618
$ git checkout pull/3618
Update a local copy of the PR:
$ git checkout pull/3618
$ git pull https://git.openjdk.org/jdk17u-dev.git pull/3618/head
Using Skara CLI tools
Checkout this PR locally:
$ git pr checkout 3618
View PR using the GUI difftool:
$ git pr show -t 3618
Using diff file
Download this PR as a diff file:
https://git.openjdk.org/jdk17u-dev/pull/3618.diff
Using Webrev
Link to Webrev Comment