RFR: 8320198: some reference processing tests don't wait long enough for reference processing to complete

2023-12-04 Thread Tom Rodriguez
This slightly increases the wait for reference processing to complete to accommodate long Xcomp compile times. Testing is underway. - Commit messages: - 8320198: some reference processing tests don't wait long enough for reference processing to complete Changes: https://git.openj

Re: RFR: 8320198: some reference processing tests don't wait long enough for reference processing to complete

2023-12-04 Thread Joe Darcy
On Mon, 4 Dec 2023 17:46:18 GMT, Tom Rodriguez wrote: > This slightly increases the wait for reference processing to complete to > accommodate long Xcomp compile times. Testing is underway. test/jdk/java/lang/Object/FinalizationOption.java line 92: > 90: System.gc(); > 91:

Re: RFR: 8320198: some reference processing tests don't wait long enough for reference processing to complete

2023-12-04 Thread Tom Rodriguez
On Mon, 4 Dec 2023 20:51:20 GMT, Joe Darcy wrote: >> This slightly increases the wait for reference processing to complete to >> accommodate long Xcomp compile times. Testing is underway. > > test/jdk/java/lang/Object/FinalizationOption.java line 92: > >> 90: System.gc(); >> 91:

Re: RFR: 8320198: some reference processing tests don't wait long enough for reference processing to complete

2023-12-04 Thread Joe Darcy
On Mon, 4 Dec 2023 21:30:44 GMT, Tom Rodriguez wrote: > I guess that sort of makes sense. The default timeout factor is 4 I think so > I guess I descale the existing total wait of 1000 ms so that the total wait > is now Utils.adjustTimeout(250)? Maybe others might offer an opinion? Since > the

Re: RFR: 8320198: some reference processing tests don't wait long enough for reference processing to complete

2023-12-06 Thread Jaikiran Pai
On Mon, 4 Dec 2023 17:46:18 GMT, Tom Rodriguez wrote: > This slightly increases the wait for reference processing to complete to > accommodate long Xcomp compile times. Testing is underway. test/jdk/java/lang/Object/FinalizationOption.java line 89: > 87: static boolean checkFinalizerCalle

Re: RFR: 8320198: some reference processing tests don't wait long enough for reference processing to complete

2023-12-07 Thread Tom Rodriguez
On Thu, 7 Dec 2023 05:18:26 GMT, Jaikiran Pai wrote: >> This slightly increases the wait for reference processing to complete to >> accommodate long Xcomp compile times. Testing is underway. > > test/jdk/java/lang/Object/FinalizationOption.java line 89: > >> 87: static boolean checkFinaliz