looks ok to me.
Regards
Prasanta
On 1/6/2017 2:11 PM, Prem Balakrishnan wrote:
Hi Prasanta,
On Ubuntu 16.04 test is taking 21350ms instead of 20000ms.
Removed manual timeout and made test rely on jtreg timeout.
http://cr.openjdk.java.net/~pkbalakr/8172009/webrev.02/
<http://cr.openjdk.java.net/%7Epkbalakr/8172009/webrev.02/>
Regards,
Prem
*From:*Prasanta Sadhukhan
*Sent:* Wednesday, January 04, 2017 4:36 PM
*To:* Ajit Ghaisas; Prem Balakrishnan; awt-dev@openjdk.java.net
*Subject:* Re: Review Request: JDK-8172009 [TEST_BUG] increase timeout
in java/awt/print/PaintSetEnabledDeadlock/PaintSetEnabledDeadlock.java
And also, I think this test has nothing to do with "print". It should
be moved to java/awt/Paint folder, I guess.
Do you have an idea why it is taking more than 20 secs to complete,
which api is taking so much time? Is it on all platform, or some specific?
Regards
Prasanta
On 1/4/2017 4:20 PM, Ajit Ghaisas wrote:
Change looks fine.
Some correction can be done by replacing wildcard imports by
specific imports.
Regards,
Ajit
*From:*Prem Balakrishnan
*Sent:* Wednesday, January 04, 2017 4:09 PM
*To:* Ajit Ghaisas; Prasanta Sadhukhan; awt-dev@openjdk.java.net
<mailto:awt-dev@openjdk.java.net>
*Subject:* Review Request: JDK-8172009 [TEST_BUG] increase timeout
in java/awt/print/PaintSetEnabledDeadlock/PaintSetEnabledDeadlock.java
Hi*,*
Please review fix for JDK 9,
*Bug:*https://bugs.openjdk.java.net/browse/JDK-8172009
*Webrev:*http://cr.openjdk.java.net/~pkbalakr/8172009/webrev.00/
<http://cr.openjdk.java.net/%7Epkbalakr/8172009/webrev.00/>
Test fails because it was taking more than 20 seconds to
complete, updated timeout to 30 seconds.
Regards,
Prem
