RFR: 8323659: LinkedTransferQueue add and put methods call overridable offer

Viktor Klang vklang at openjdk.org
Fri Jan 12 11:54:19 UTC 2024


On Fri, 12 Jan 2024 11:22:13 GMT, Chris Hegarty <chegar at openjdk.org> wrote:

>>> A process-related comment: this PR is against mainline, but the issue's Fix Version/s is 22. This will create a bit of a mess if integrated.
>> 
>> Thanks for the reminder Pavel.  If accepted, then the change will be applicable to 23, 22, and 21. 0.2.  I'll fix up and retarget the appropriate branches, repos, once we agree a way forward.
>
>> this PR is against mainline, but the issue's Fix Version/s is 22.
> 
> I updated the fix version in JIRA, and followed the process as outlined in https://mail.openjdk.org/pipermail/jdk-dev/2023-December/008560.html

@ChrisHegarty My 2¢—this change could impact existing classes which have extended LTQ and knowingly, or unknowingly, presume that add and put delegate to offer, which would (silently) break those classes when running on a newer JDK (presuming this change gets merged).

I think the change is in principle correct, with that said, I want to make sure that the change is worth the potential compatibility risk (if any).

-------------

PR Comment: https://git.openjdk.org/jdk/pull/17393#issuecomment-1888966783


More information about the core-libs-dev mailing list