Hi,

The pid (though os specific) is available from the Process API and so adding it to toString
does not expose any additional implementation specific information.

Getting accurate values for exitcode and running status would require a native call
on windows but might be ok only for debugging.

$.02, Roger


On 4/25/2016 3:49 PM, Martin Buchholz wrote:
This is a reasonable request.

But if you put platform-specific information into the String, then
people will start treating that as an API (parsing the string), which
is undesirable.

On Mon, Apr 25, 2016 at 12:33 PM, Steven Schlansker
<stevenschlans...@gmail.com> wrote:
Hi core-libs-dev,

I recently was diagnosing a problem relating to an external program invoked via
the ProcessBuilder API.

The returned Process is an instance of java.lang.UNIXProcess, which does
not have a toString method.

While I understand that the concepts of "pid" etc are not cross-platform,
it might be useful to operators to have the toString method
dump them as diagnostic information.

Imagine if instead of

"java.lang.UNIXProcess@<hashCode>"

you could see

"UNIXProcess[running=true, pid=1234]"
"UNIXProcess[running=false, exitCode=127]"

This seems like it would be a fairly trivial change that could help some
operator down the road.

Is this a reasonable request?  Thanks,
Steven


Reply via email to