On Sat, 8 Jun 2024 02:21:34 GMT, David Holmes wrote:
>> Allow warning messages such as the following to appear in stderr:
>>
>> Java HotSpot(TM) 64-Bit Server VM warning: Temporarily processing option
>> UseNotificationThread; support is scheduled for removal in 24.0
>>
>> Tested by running CI
On Fri, 7 Jun 2024 10:31:22 GMT, Jaikiran Pai wrote:
> Can I please get a review of this test-only change which fixes an issue that
> was introduced due to the refactoring that we did in
> https://bugs.openjdk.org/browse/JDK-8333130? This PR addresses the failure
> reported in https://bugs.ope
On Fri, 7 Jun 2024 19:56:03 GMT, Chris Plummer wrote:
> Allow warning messages such as the following to appear in stderr:
>
> Java HotSpot(TM) 64-Bit Server VM warning: Temporarily processing option
> UseNotificationThread; support is scheduled for removal in 24.0
>
> Tested by running CI tier
On Fri, 7 Jun 2024 19:56:03 GMT, Chris Plummer wrote:
> Allow warning messages such as the following to appear in stderr:
>
> Java HotSpot(TM) 64-Bit Server VM warning: Temporarily processing option
> UseNotificationThread; support is scheduled for removal in 24.0
>
> Tested by running CI tier
On Fri, 7 Jun 2024 19:56:03 GMT, Chris Plummer wrote:
> Allow warning messages such as the following to appear in stderr:
>
> Java HotSpot(TM) 64-Bit Server VM warning: Temporarily processing option
> UseNotificationThread; support is scheduled for removal in 24.0
>
> Tested by running CI tier
Allow warning messages such as the following to appear in stderr:
Java HotSpot(TM) 64-Bit Server VM warning: Temporarily processing option
UseNotificationThread; support is scheduled for removal in 24.0
Tested by running CI tier5, which reproduced the issue.
-
Commit messages:
- A
On Thu, 6 Jun 2024 02:12:11 GMT, Alex Menkov wrote:
> The fix updates com/sun/tools/attach/BasicTests.java to listen and connect
> using loopback addresses
>
> Testing: run the test on all Oracle-supported platforms
This pull request has now been integrated.
Changeset: 17bd483f
Author:Ale
On Fri, 7 Jun 2024 10:31:22 GMT, Jaikiran Pai wrote:
> Can I please get a review of this test-only change which fixes an issue that
> was introduced due to the refactoring that we did in
> https://bugs.openjdk.org/browse/JDK-8333130? This PR addresses the failure
> reported in https://bugs.ope
On Tue, 4 Jun 2024 20:51:52 GMT, Cesar Soares Lucas wrote:
> Please, consider this patch to remove unused methods from the code base. To
> the best of my knowledge, these methods are only defined but never used.
>
> Here is a list with names of delete methods:
> https://gist.github.com/JohnTor
On Tue, 4 Jun 2024 20:51:52 GMT, Cesar Soares Lucas wrote:
> Please, consider this patch to remove unused methods from the code base. To
> the best of my knowledge, these methods are only defined but never used.
>
> Here is a list with names of delete methods:
> https://gist.github.com/JohnTor
On Thu, 6 Jun 2024 02:12:11 GMT, Alex Menkov wrote:
> The fix updates com/sun/tools/attach/BasicTests.java to listen and connect
> using loopback addresses
>
> Testing: run the test on all Oracle-supported platforms
Marked as reviewed by lmesnik (Reviewer).
-
PR Review: https://g
On Fri, 7 Jun 2024 07:29:39 GMT, SendaoYan wrote:
>> Hi all,
>> This PR several extra empty spaces and extra empty lines in several
>> Makefiles. It's trivial fix, no risk.
>>
>> Thanks.
>
> SendaoYan has updated the pull request incrementally with one additional
> commit since the last revi
On Tue, 4 Jun 2024 07:47:46 GMT, SendaoYan wrote:
> Hi all,
> This PR several extra empty spaces and extra empty lines in several
> Makefiles. It's trivial fix, no risk.
>
> Thanks.
This pull request has now been integrated.
Changeset: d130d2f4
Author:SendaoYan
Committer: Julian Waters
On Fri, 7 Jun 2024 13:01:15 GMT, SendaoYan wrote:
>> As confusing as they are, unfortunately GitHub UI does not render extra
>> trailing newlines. This is the only one I could find with grepWin.
>
> I find the extra trailing newlines through below shell command:
>
> for i in `find . -iname "Mak
On Tue, 4 Jun 2024 20:51:52 GMT, Cesar Soares Lucas wrote:
> Please, consider this patch to remove unused methods from the code base. To
> the best of my knowledge, these methods are only defined but never used.
>
> Here is a list with names of delete methods:
> https://gist.github.com/JohnTor
On Fri, 7 Jun 2024 12:53:46 GMT, Chen Liang wrote:
>> No, it's an extra newline. A file should end with a newline but one is
>> enough.
>
> As confusing as they are, unfortunately GitHub UI does not render extra
> trailing newlines. This is the only one I could find with grepWin.
I find the ex
> Please review this enhancement to the container detection code which allows
> it to figure out whether the JVM is actually running inside a container
> (`podman`, `docker`, `crio`), or with some other means that enforces
> memory/cpu limits by means of the cgroup filesystem. If neither of thos
On Fri, 7 Jun 2024 12:47:39 GMT, Erik Joelsson wrote:
>> test/jdk/java/rmi/reliability/benchmark/bench/Makefile line 50:
>>
>>> 48: clean:
>>> 49: rm -f *.class .classes
>>> 50:
>>
>> Hmm, shouldn't this newline at EOF be kept? Asking @ all the people who've
>> reviewed this so far, n
On Fri, 7 Jun 2024 12:37:48 GMT, Julian Waters wrote:
>> SendaoYan has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> delete extra empty trailing blank line in
>> test/jdk/java/rmi/reliability/benchmark/bench/Makefile
>
> test/jdk/java/rmi
On Fri, 7 Jun 2024 07:29:39 GMT, SendaoYan wrote:
>> Hi all,
>> This PR several extra empty spaces and extra empty lines in several
>> Makefiles. It's trivial fix, no risk.
>>
>> Thanks.
>
> SendaoYan has updated the pull request incrementally with one additional
> commit since the last revi
On Fri, 7 Jun 2024 07:26:39 GMT, SendaoYan wrote:
>> test/jdk/java/rmi/reliability/benchmark/bench/rmi/Makefile line 1:
>>
>>> 1: #
>>
>> This file change is dubious:
>> 1. It does not have any trailing whitespace that can fail the skara checks.
>> 2. If the duplicate blank lines in the end of
On Fri, 7 Jun 2024 07:29:39 GMT, SendaoYan wrote:
>> Hi all,
>> This PR several extra empty spaces and extra empty lines in several
>> Makefiles. It's trivial fix, no risk.
>>
>> Thanks.
>
> SendaoYan has updated the pull request incrementally with one additional
> commit since the last revi
On Fri, 7 Jun 2024 07:29:39 GMT, SendaoYan wrote:
>> Hi all,
>> This PR several extra empty spaces and extra empty lines in several
>> Makefiles. It's trivial fix, no risk.
>>
>> Thanks.
>
> SendaoYan has updated the pull request incrementally with one additional
> commit since the last revi
On Tue, 4 Jun 2024 20:51:52 GMT, Cesar Soares Lucas wrote:
> Please, consider this patch to remove unused methods from the code base. To
> the best of my knowledge, these methods are only defined but never used.
>
> Here is a list with names of delete methods:
> https://gist.github.com/JohnTor
On Tue, 4 Jun 2024 20:51:52 GMT, Cesar Soares Lucas wrote:
> Please, consider this patch to remove unused methods from the code base. To
> the best of my knowledge, these methods are only defined but never used.
>
> Here is a list with names of delete methods:
> https://gist.github.com/JohnTor
On Tue, 4 Jun 2024 20:51:52 GMT, Cesar Soares Lucas wrote:
> Please, consider this patch to remove unused methods from the code base. To
> the best of my knowledge, these methods are only defined but never used.
>
> Here is a list with names of delete methods:
> https://gist.github.com/JohnTor
On Fri, 7 Jun 2024 07:29:39 GMT, SendaoYan wrote:
>> Hi all,
>> This PR several extra empty spaces and extra empty lines in several
>> Makefiles. It's trivial fix, no risk.
>>
>> Thanks.
>
> SendaoYan has updated the pull request incrementally with one additional
> commit since the last revi
Can I please get a review of this test-only change which fixes an issue that
was introduced due to the refactoring that we did in
https://bugs.openjdk.org/browse/JDK-8333130? This PR addresses the failure
reported in https://bugs.openjdk.org/browse/JDK-8333756.
The `NativeMethodPrefixApp` test
On Thu, 6 Jun 2024 17:49:08 GMT, Chen Liang wrote:
>> SendaoYan has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> delete extra empty trailing blank line in
>> test/jdk/java/rmi/reliability/benchmark/bench/Makefile
>
> test/jdk/java/rmi/re
> Hi all,
> This PR several extra empty spaces and extra empty lines in several
> Makefiles. It's trivial fix, no risk.
>
> Thanks.
SendaoYan has updated the pull request incrementally with one additional commit
since the last revision:
delete extra empty trailing blank line in
test/jdk/j
30 matches
Mail list logo
