[java] use the java process builder to run external processes

[joerg1985]

Description

Replaced commons-exec with the java process builder to get rid of the dependency.

Motivation and Context

With Java 11 we now have all we need to implement the same functionality without a dependency.

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist

• I have read the contributing document.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.

[codecov-commenter]

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (0f8e018) 56.51% compared to head (541f72c) 56.51%.
Report is 1 commits behind head on trunk.

❗ Current head 541f72c differs from pull request most recent head 2877334. Consider uploading reports for the commit 2877334 to get more accurate results

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@           Coverage Diff           @@
##            trunk   #12898   +/-   ##
=======================================
  Coverage   56.51%   56.51%           
=======================================
  Files          86       86           
  Lines        5255     5255           
  Branches      187      187           
=======================================
  Hits         2970     2970           
  Misses       2098     2098           
  Partials      187      187           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[titusfortner]

Oh shoot, I was going to merge this before 4.14. :(

[diemol]

Thank you, @joerg1985!

@mykola-mokhnach @SrinivasanTarget @valfirst, does this affect Appium?

[mykola-mokhnach]

Thank you, @joerg1985!

@mykola-mokhnach @SrinivasanTarget @valfirst, does this affect Appium?

yes it does: appium/java-client#2036 (comment)

That is why I have requested this feature

[diemol]

Thank you, @joerg1985!
@mykola-mokhnach @SrinivasanTarget @valfirst, does this affect Appium?

yes it does: appium/java-client#2036 (comment)

That is why I have requested this feature

OK, so you are already checking this. We should be able to merge this anytime, @joerg1985.

[valfirst]

@mykola-mokhnach it seems you mixed up PRs, this one about process management

@diemol it affects Appium, but I'm not sure how much, anyway once SNAPSHOT is published, it can be tested