RE: RFR : JDK-8175542 - JMX: Not enough JDP packets received

2018-01-14 Thread Amit Sapre
t; Thanks Harsha for the inputs. > > > > I made changes in this webrev : > > http://cr.openjdk.java.net/~asapre/webrev/2018/JDK- > 8175542/webrev.01/ > > > > Thanks, > > > > Amit > > > > *From:*Harsha Wardhana B > > *Sent:* Wednesd

Re: RFR : JDK-8175542 - JMX: Not enough JDP packets received

2018-01-11 Thread Harsha Wardhana B
JDK-8175542/webrev.01/> Thanks, Amit *From:*Harsha Wardhana B *Sent:* Wednesday, December 13, 2017 5:32 PM *To:* serviceability-dev@openjdk.java.net *Subject:* Re: RFR : JDK-8175542 - JMX: Not enough JDP packets received Hi Amit, Increasing the timeout 'TIME_OUT_FACTOR' will increase both

Re: RFR : JDK-8175542 - JMX: Not enough JDP packets received

2018-01-10 Thread David Holmes
te for now. Thanks, David Thanks, Amit *From:*Amit Sapre *Sent:* Wednesday, January 03, 2018 5:54 PM *To:* Harsha wardhana B; serviceability-dev@openjdk.java.net *Subject:* RE: RFR : JDK-8175542 - JMX: Not enough JDP packets received Thanks Harsha for the inputs. I made changes in this webr

RE: RFR : JDK-8175542 - JMX: Not enough JDP packets received

2018-01-10 Thread Amit Sapre
Hello, Ping. Can somebody review this code changes ? Thanks, Amit From: Amit Sapre Sent: Wednesday, January 03, 2018 5:54 PM To: Harsha wardhana B; serviceability-dev@openjdk.java.net Subject: RE: RFR : JDK-8175542 - JMX: Not enough JDP packets received Thanks Harsha

RE: RFR : JDK-8175542 - JMX: Not enough JDP packets received

2018-01-03 Thread Amit Sapre
- JMX: Not enough JDP packets received Hi Amit, Increasing the timeout 'TIME_OUT_FACTOR' will increase both socket and test-case timeout. The test passed as can be seen from the log, but because of the race-condition: hasTestLivedLongEnough() checked before shouldContinue(), the test

Re: RFR : JDK-8175542 - JMX: Not enough JDP packets received

2017-12-13 Thread Harsha Wardhana B
Hi Amit, Increasing the timeout 'TIME_OUT_FACTOR' will increase both socket and test-case timeout. The test passed as can be seen from the log, but because of the race-condition: hasTestLivedLongEnough() checked before shouldContinue(), the test was declared failed because of time-out. One