RFR: 8303392: Runtime.exec and ProcessBuilder.start should use System logger [v3]
Roger Riggs
rriggs at openjdk.org
Fri Mar 10 15:15:12 UTC 2023
On Thu, 9 Mar 2023 20:24:11 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:
>> Roger Riggs has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Revise logging of ProcessBuilder.start to support DEBUG and TRACE logging.
>> Revise the implNote to include a warning about logging security sensistive information.
>> DEBUG logs only the command, directory, stack trace, and pid.
>> TRACE additionally logs arguments.
>
> test/jdk/java/lang/ProcessBuilder/ProcessStartLoggingTest.java line 81:
>
>> 79: File nullDirectory = null;
>> 80: File thisDirectory = new File(".");
>> 81:
>
> I started to heavily use different `@test` sections with speaking names instead of running a bunch of test in sequence; the advantage is better parallelization of tests, that I can omit printing out the test name manually (if the test name itself is speaking), and the ability to start individual tests manually. It does come with lots of test sections though.
Speakable names can be a plus but I'll stick to the parameterized version for now.
Are you refering to JUnit sections or Jtreg sections?
-------------
PR: https://git.openjdk.org/jdk/pull/12862
More information about the core-libs-dev
mailing list