Re: RR(S): JDK-8004213 jdp packet need pid, broadcast interval and rmi server hostname fields.

2013-10-11 Thread Peter Allwin
Hi Dmitry, Looks good, thanks for fixing this! Regards, /peter On Oct 10, 2013, at 8:33 PM, Dmitry Samersoff wrote: > Hi Everyone, > > This fix covers > > JDK JDK-8004213 JDP packets should contain PID > JDK-8004215 JDP packets should contain the configured broadcast interval > JDK-8007071

Re: RR(S): JDK-8004213 jdp packet need pid, broadcast interval and rmi server hostname fields.

2013-10-10 Thread Staffan Larsen
Dmitry, This looks good, just two small nits: test/sun/management/jdp/JdpClient.java: 56: "Packet error fileld " -> "Packet error: field " test/sun/management/jdp/JdpClient.java: 68-74: missing space after the comma Thanks, /Staffan On 10 okt 2013, at 20:33, Dmitry Samersoff wrote: > Hi Ev

RR(S): JDK-8004213 jdp packet need pid, broadcast interval and rmi server hostname fields.

2013-10-10 Thread Dmitry Samersoff
Hi Everyone, This fix covers JDK JDK-8004213 JDP packets should contain PID JDK-8004215 JDP packets should contain the configured broadcast interval JDK-8007071 JDP does not work well with java.rmi.server.hostname overrides. and add three extra fields to JDP packet: PID, rmi server name, broadca