Re: RFR: 8266646: Add more diagnostic to java/lang/System/LoggerFinder/modules [v3]

2021-05-06 Thread Lance Andersen 
On Thu, 6 May 2021 16:57:12 GMT, Daniel Fuchs  wrote:

>> Hi, please find here a trivial test change that adds some diagnostic (time 
>> stamps) to the LoggerFinder/modules subprocess test logs.
>
> Daniel Fuchs has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   removed unused import statement

Marked as reviewed by lancea (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/3904

Re: RFR: 8266646: Add more diagnostic to java/lang/System/LoggerFinder/modules [v3]

2021-05-06 Thread Brian Burkhalter 
On Thu, 6 May 2021 16:57:12 GMT, Daniel Fuchs  wrote:

>> Hi, please find here a trivial test change that adds some diagnostic (time 
>> stamps) to the LoggerFinder/modules subprocess test logs.
>
> Daniel Fuchs has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   removed unused import statement

Marked as reviewed by bpb (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/3904

Re: RFR: 8266646: Add more diagnostic to java/lang/System/LoggerFinder/modules [v3]

2021-05-06 Thread Iris Clark 
On Thu, 6 May 2021 16:57:12 GMT, Daniel Fuchs  wrote:

>> Hi, please find here a trivial test change that adds some diagnostic (time 
>> stamps) to the LoggerFinder/modules subprocess test logs.
>
> Daniel Fuchs has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   removed unused import statement

Marked as reviewed by iris (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/3904

Re: RFR: 8266646: Add more diagnostic to java/lang/System/LoggerFinder/modules [v3]

2021-05-06 Thread Naoto Sato 
On Thu, 6 May 2021 16:57:12 GMT, Daniel Fuchs  wrote:

>> Hi, please find here a trivial test change that adds some diagnostic (time 
>> stamps) to the LoggerFinder/modules subprocess test logs.
>
> Daniel Fuchs has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   removed unused import statement

Marked as reviewed by naoto (Reviewer).

-

PR: https://git.openjdk.java.net/jdk/pull/3904

Re: RFR: 8266646: Add more diagnostic to java/lang/System/LoggerFinder/modules [v3]

2021-05-06 Thread Daniel Fuchs 
> Hi, please find here a trivial test change that adds some diagnostic (time 
> stamps) to the LoggerFinder/modules subprocess test logs.

Daniel Fuchs has updated the pull request incrementally with one additional 
commit since the last revision:

  removed unused import statement

-

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/3904/files
  - new: https://git.openjdk.java.net/jdk/pull/3904/files/3e6b436c..c101347a

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3904&range=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3904&range=01-02

  Stats: 4 lines in 4 files changed: 0 ins; 4 del; 0 mod
  Patch: https://git.openjdk.java.net/jdk/pull/3904.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/3904/head:pull/3904

PR: https://git.openjdk.java.net/jdk/pull/3904