RFR 9: 8034903: Add Logging of Process.start arguments and resulting pid
pbenedict at apache.org
Thu Feb 13 21:50:54 UTC 2014
Roger, I only have two suggested improvements:
1) solaris/../ProcessImpl.java should use Objects.toString() rather than
the tenary operator for a string choice. You did this alright in
2) /windows/../ProcessImpl.java doesn't need to specify "null" for
Objects.toString() since that is its default output for null input.
On Thu, Feb 13, 2014 at 3:26 PM, roger riggs <roger.riggs at oracle.com> wrote:
> Having folks stumbling over process creation and problems of quoting,
> especially on windows, it seems useful to log the native commands and
> They are proposed to be logged using the PlatformLogger at Level.FINE
> which will not be logged by default. The environment is useful in some
> but verbose, that it should be Logged at FINER. The pid of the spawned
> logged as well for traceability.
> Please take a look at this first draft and comment on whether it is useful,
> a good idea and any improvements needed.
> Thanks, Roger
More information about the core-libs-dev