On Tue, 9 Jul 2024 08:39:39 GMT, Jaikiran Pai <j...@openjdk.org> wrote:
> Can I please get a review of this change which removes an incorrect entry > from `ProblemList-Virtual.txt`? > > As noted in https://bugs.openjdk.org/browse/JDK-8335966 this entry is a left > over which went unnoticed when we did the changes in > https://bugs.openjdk.org/browse/JDK-8333130. > > The refactored test `NativeMethodPrefixApp` has been functioning without the > errors for which `NativeMethodPrefixAgent` was previously problem listed. So > I haven't replaced that problem list entry and instead have just removed it. > > `NativeMethodPrefixApp` is known to run into an intermittent timeout with > virtual threads and that's tracked in > https://bugs.openjdk.org/browse/JDK-8334167 and I think it's not problem > listing worthy given the relatively low frequency of such timeouts. > Furthermore, in the coming days, I plan to propose a change to this > `NativeMethodPrefixApp` test which should fix the timeout failures. > > To verify that this removal doesn't cause any unexpected issues, I've > triggered a CI run of the relevant tier. Thank you Kevin and Alex for the reviews. tier5, where this problem listing file is in use, completed without issues. I will go ahead and integrate this in the next few hours. ------------- PR Comment: https://git.openjdk.org/jdk/pull/20094#issuecomment-2219305134