RFR: 8252767: URLConnection.setRequestProperty throws IllegalAccessError [v4]

[Jaikiran Pai]

On Mon, 7 Sep 2020 09:31:02 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> I see Jaikiran has used "/integrate" but I don't think the review is complete yet. The change may be trivial but it's
>> really important to give others a chance to comment if they wish.
>
>> I see Jaikiran has used "/integrate" but I don't think the review is complete yet. The change may be trivial but it's
>> really important to give others a chance to comment if they wish.
> 
> Sorry, I'm new this whole flow. I saw the message from that bot asking me to issue that command. I then went into the
> "contributing" guidelines to see if there was anything else needed to be done but I didn't see anything specific, so
> went ahead and issued the `/integrate`.   I guess if I force push on to this PR, with that TestNG change that you
> recommended, perhaps the review process will put this PR into the original state which it was before the `/integrate`
> command?

Slightly OT - but I think this `/integrate` command shouldn't actually be in the hands of the contributor and instead
should be decided and issued by "committers" or "sponsors". I don't really see how the contributor/author can
realistically decide when it's the right time to intiate a `/integrate`

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

PR: https://git.openjdk.java.net/jdk/pull/26