JSON metrics report path supports variables substitution

[pan3793]

Why are the changes needed?

This PR enhances kyuubi.metrics.json.location to support variable substitution: {{KYUUBI_HOME}}, {{KYUUBI_WORK_DIR_ROOT}}, and changes the default value to {{KYUUBI_HOME}}/metrics - I suppose it is a more reasonable default value than {{KYUUBI_WORK_DIR_ROOT}}/metrics, given others dirs also under KYUUBI_HOME by default

export KYUUBI_LOG_DIR="${KYUUBI_LOG_DIR:-"${KYUUBI_HOME}/logs"}"
export KYUUBI_PID_DIR="${KYUUBI_PID_DIR:-"${KYUUBI_HOME}/pid"}"
export KYUUBI_WORK_DIR_ROOT="${KYUUBI_WORK_DIR_ROOT:-"${KYUUBI_HOME}/work"}"

To make things more consistent, variable substitution behavior is also updated for kyuubi.metadata.store.jdbc.url

How was this patch tested?

Manually tested by

bin/kyuubi run \
  --conf kyuubi.metrics.reporters=JSON \
  --conf kyuubi.metrics.json.location={{KYUUBI_HOME}}/metrics

bin/kyuubi run \
  --conf kyuubi.metrics.reporters=JSON \
  --conf kyuubi.metrics.json.location={{KYUUBI_WORK_DIR_ROOT}}/metrics

and checks the metrics JSON file.

Was this patch authored or co-authored using generative AI tooling?

No.

[codecov-commenter]

Codecov Report

❌ Patch coverage is 0% with 23 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.00%. Comparing base (69e8e95) to head (71e4b28).

Files with missing lines	Patch %	Lines
...ommon/src/main/scala/org/apache/kyuubi/Utils.scala	0.00%	12 Missing ⚠️
...i/server/metadata/jdbc/JDBCMetadataStoreConf.scala	0.00%	3 Missing ⚠️
...rg/apache/kyuubi/metrics/JsonReporterService.scala	0.00%	2 Missing ⚠️
.../scala/org/apache/kyuubi/metrics/MetricsConf.scala	0.00%	2 Missing ⚠️
...n/scala/org/apache/kyuubi/engine/ProcBuilder.scala	0.00%	2 Missing ⚠️
...rg/apache/kyuubi/zookeeper/EmbeddedZookeeper.scala	0.00%	2 Missing ⚠️

Additional details and impacted files

@@          Coverage Diff           @@
##           master   #7310   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files         698     698           
  Lines       43649   43654    +5     
  Branches     5894    5896    +2     
======================================
- Misses      43649   43654    +5     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[wForget]

Thanks @pan3793 , LGTM

[pan3793]

thanks, merging to master