vorburger/MariaDB4j — comment on commit 49462a7e49310d87c3926376fff974dda378fcf8
File: src/main/java/ch/vorburger/exec/RunningProcess.java line 43
By szermatt:

It bothers me a bit that a constructor would start a new process.
You'll tell me that it's actually done by the command exec() on the Builder, but package local isn't much of a protection.

I would be safer if either:
- this class and its builder were put together into one file, so you could have a private constructor. The builder could call start()
- the constructor was made private and callable only from a static method that would build the RunningProcess and start it

In both case, you prevent code even in this package from creating new instances or subclassing, also it's clearer that something happens.