Fixing StartZooKeeper Command

[xiangfu0]

1. Allow user to customize zk data dir
2. Don't delete zk data dir after process shutdown. (Default zk data dir is under java tmp, which will be auto recycled)

[codecov-io]

Codecov Report

Merging #5295 into master will decrease coverage by 9.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5295      +/-   ##
==========================================
- Coverage   66.09%   56.95%   -9.15%     
==========================================
  Files        1067     1070       +3     
  Lines       54543    54598      +55     
  Branches     8130     8134       +4     
==========================================
- Hits        36050    31094    -4956     
- Misses      15848    21075    +5227     
+ Partials     2645     2429     -216     

Impacted Files	Coverage Δ
...a/org/apache/pinot/minion/metrics/MinionMeter.java	0.00% <0.00%> (-100.00%)	⬇️
.../apache/pinot/common/metrics/BrokerQueryPhase.java	0.00% <0.00%> (-100.00%)	⬇️
.../apache/pinot/minion/metrics/MinionQueryPhase.java	0.00% <0.00%> (-100.00%)	⬇️
...he/pinot/core/query/reduce/ComparisonFunction.java	0.00% <0.00%> (-100.00%)	⬇️
...pinot/minion/exception/TaskCancelledException.java	0.00% <0.00%> (-100.00%)	⬇️
...ot/minion/events/EventObserverFactoryRegistry.java	0.00% <0.00%> (-100.00%)	⬇️
...t/minion/executor/TaskExecutorFactoryRegistry.java	0.00% <0.00%> (-100.00%)	⬇️
...minion/executor/ConvertToRawIndexTaskExecutor.java	0.00% <0.00%> (-100.00%)	⬇️
...nion/events/DefaultMinionEventObserverFactory.java	0.00% <0.00%> (-100.00%)	⬇️
...startree/executor/StarTreeAggregationExecutor.java	0.00% <0.00%> (-100.00%)	⬇️
... and 317 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7fd2110...994cb88. Read the comment docs.