RFR: JDK-8216558: Lookup.unreflectSetter(Field) fails to throw IllegalAccessException for final fields

Joe Darcy joe.darcy at oracle.com
Tue Mar 12 03:32:43 UTC 2019


On 3/11/2019 3:49 PM, Mandy Chung wrote:
>
>
> On 3/11/19 8:48 AM, Adam Farley8 wrote:
>> Hi Mandy,
>>
>> Thank you for explaining. :)
>>
>> Unfortunately I'm only a mere Author, and I cannot submit test runs 
>> on the shared test server.
>>
>> I have run the test locally, and it passes against a patched build 
>> and fails correctly agaionst an unpatched build, so I think we're 
>> good to go.
>
> I will sponsor this patch for you when CSR is approved.
>
>> Can you tell me if there's something I can do to help move the CSR 
>> forward?
>
> No additional action from your end.  It's already in finalized state.
>
>> Also, can you tell me if an additional CSR would need to be created 
>> for other releases if this fix gets backported?
>
> Which release are you thinking to backport to?
>
> IMO I don't think this fix is critical for existing releases
> and this fix has behavioral change.


As a procedural matter, to do a CSR for multiple releases, first create 
the backport issue for the release or release train in question (e.g. 
some particular update release like "12.0.1" or "12-pool") and then 
create a CSR of the backport. This is the procedure if an update release 
train is using the CSR review process.

However, note that as this proposed changes alters a spec in the Java SE 
API, it would require a maintenance review of the Java SE spec, which is 
very unlikely to occur in an update release.

HTH,

-Joe



More information about the core-libs-dev mailing list