[Improvement-14243][Metrics] Add user tag of response time of api server

[rickchengx]

Purpose of the pull request

• close: [Improvement][Metrics] Add user tag of response time of api server #14243

Brief change log

• Add user tag of response time of api server

[codecov-commenter]

Codecov Report

Merging #14248 (4948e1b) into dev (7226edd) will increase coverage by 0.03%.
The diff coverage is 78.26%.

❗ Current head 4948e1b differs from pull request most recent head 56097d2. Consider uploading reports for the commit 56097d2 to get more accurate results

@@             Coverage Diff              @@
##                dev   #14248      +/-   ##
============================================
+ Coverage     38.35%   38.38%   +0.03%     
+ Complexity     4471     4470       -1     
============================================
  Files          1229     1229              
  Lines         42936    42892      -44     
  Branches       4765     4760       -5     
============================================
- Hits          16466    16463       -3     
+ Misses        24646    24607      -39     
+ Partials       1824     1822       -2     

Impacted Files	Coverage Δ
...er/api/service/impl/MetricsCleanUpServiceImpl.java	12.50% <0.00%> (-1.79%)	⬇️
.../server/master/runner/StateWheelExecuteThread.java	0.50% <ø> (+0.10%)	⬆️
.../server/master/runner/WorkflowExecuteRunnable.java	10.33% <ø> (+0.01%)	⬆️
...dolphinscheduler/api/metrics/ApiServerMetrics.java	84.37% <72.72%> (-5.79%)	⬇️
...e/dolphinscheduler/api/aspect/AccessLogAspect.java	86.95% <100.00%> (+0.59%)	⬆️
...inscheduler/api/service/impl/UsersServiceImpl.java	70.83% <100.00%> (+0.04%)	⬆️

... and 3 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[EricGao888]

Clean up mechanism example -

dolphinscheduler/dolphinscheduler-master/src/main/java/org/apache/dolphinscheduler/server/master/metrics/ProcessInstanceMetrics.java

Lines 93 to 101 in 5e9f1de

	public void cleanUpProcessInstanceCountMetricsByDefinitionCode(final String processDefinitionCode) {
	for (final String state : processInstanceStates) {
	final Counter counter = Metrics.globalRegistry.counter(
	"ds.workflow.instance.count",
	"state", state,
	"process.definition.code", processDefinitionCode);
	Metrics.globalRegistry.remove(counter);
	}
	}

[rickchengx]

Clean up mechanism example -

dolphinscheduler/dolphinscheduler-master/src/main/java/org/apache/dolphinscheduler/server/master/metrics/ProcessInstanceMetrics.java

Lines 93 to 101 in 5e9f1de

	public void cleanUpProcessInstanceCountMetricsByDefinitionCode(final String processDefinitionCode) {
	for (final String state : processInstanceStates) {
	final Counter counter = Metrics.globalRegistry.counter(
	"ds.workflow.instance.count",
	"state", state,
	"process.definition.code", processDefinitionCode);
	Metrics.globalRegistry.remove(counter);
	}
	}

Thanks for your comment, I've cleaned up the response time metrics when user is deleted.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
3 Code Smells

82.6% Coverage
0.0% Duplication