On Tue, 27 Feb 2024 17:24:03 GMT, Chad Rakoczy <d...@openjdk.org> wrote:

> [JDK-8299677](https://bugs.openjdk.java.net/browse/JDK-8299677) fixes a bug 
> with Formatter.format taking a long time when there is a lot of padding. This 
> test runs Formatter.format with very large padding. Test fails before 
> [JDK-8299677](https://bugs.openjdk.java.net/browse/JDK-8299677) and passes 
> after.
> 
> Timeout for the test was set to 10 seconds. Test passes locally with as low 
> as 1 (after [JDK-8299677](https://bugs.openjdk.java.net/browse/JDK-8299677)) 
> and fails as high as 120 (before 
> [JDK-8299677](https://bugs.openjdk.java.net/browse/JDK-8299677)) so it should 
> be consistent.

"8326718: Test java/util/Formatter/Padding.java _does_ timeout on large inputs 
_before_ fix in JDK-8299677"
or
"8326718: Test java/util/Formatter/Padding.java _does not_ timeout on large 
inputs _after_ JDK-8299677"

To me, 
"8326718: Test java/util/Formatter/Padding.java does not timeout on large 
inputs before fix in JDK-8299677"
as you propose, sounds like the fix JDK-8299677 didn't serve its intended 
purpose.

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

PR Comment: https://git.openjdk.org/jdk/pull/18033#issuecomment-1967456018

Reply via email to