jmx-dev [8] RFR(S): 8147857: RMIConnector logs attribute names incorrectly

[Andrew Hughes]

----- Original Message -----
> 
> On 22/01/16 15:58, Andrew Hughes wrote:
> >> >Yes - please fix this issue in JDK 9 first as per rule 1:
> >> >http://openjdk.java.net/projects/jdk8u/groundrules.html
> >> >
> >> >In a nutshell, produce a JDK 9 webrev, send it to
> >> >serviceability-dev at openjdk.java.net  for review and once reviewed, push
> >> >to jdk9/dev forest. Once done, you can then submit a JDK 8u integration
> >> >approval request.
> > I've applied Severin's patch to 9 and it applies as-is, once shuffled.
> > Are the existing reviews sufficient? Jaroslav, Daniel & I have all OKed
> > it already.
> >
> > I do think the copyright notice should read 'Red Hat' and I've raised
> > this with Severin.
> I'm not sure. Everyone reviews and fixes in JDK 9 first and then
> backports where necessary. Since this was also reviewed on
> serviceability-dev, I think you might not need another round of reviews.
> 
> regards,
> Sean.
> 
> 

Ok, I pushed the change:

http://hg.openjdk.java.net/jdk9/dev/jdk/rev/452162be5da9

I know it's not the ideal way around to do this, but if it had been a case
of three people reviewing it in 9, and it applying to 8 with an unshuffle,
a re-review wouldn't have been required. Hence, I'm taking it as the same
this way around, but in future, Severin, please always post to 9 first
unless it really is inapplicable there.

Also, the actual fix is just a one-line typo correction :-) Most of the
changeset is a test case for it, and I built and ran the test before
pushing.

Thanks,
-- 
Andrew :)

Senior Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

PGP Key: ed25519/35964222 (hkp://keys.gnupg.net)
Fingerprint = 5132 579D D154 0ED2 3E04  C5A0 CFDA 0F9B 3596 4222