fix(server): Filter dynamice path(PUT/GET/DELETE) with params cause OOM

[JackyYangPassion]

Purpose of the PR

• fix [Bug] AccessLogFilter for path matrics make Server OOM #2566

Main Changes

Verifying these changes

• Trivial rework / code cleanup without any test coverage. (No Need)
• Already covered by existing tests, such as (please modify tests here).
• Need tests and can be verified as follows:
  • xxx

Does this PR potentially affect the following parts?

• Dependencies (add/update license info & regenerate_dependencies)
• Modify configurations
• The public API
• Other affects (typed here)
• Nope

Documentation Status

• Doc - TODO
• Doc - Done
• Doc - No Need

[imbajin]

@SunnyBoy-WYH could u take a look for it~

[SunnyBoy-WYH]

can we merge the requestContext.getUriInfo()?

URI uri = requestContext.getUriInfo().getRequestUri();
String path = uri.getRawPath();
String method = requestContext.getMethod();
UriInfo uriInfo = requestContext.getUriInfo();
String path = normalizePath(uriInfo.getPath(),method);
String metricsName = join(path, method);

[SunnyBoy-WYH]

Let’s not talk about the oom for now. for HISTOGRAM metric，seems path without param will be better.? @JackyYangPassion @imbajin

[SunnyBoy-WYH]

As a consult, Do you have oom system log and ensure the oom due to this map?

[JackyYangPassion]

As a consult, Do you have oom system log and ensure the oom due to this map?

Yes
Correction approach:

1. Determine if there are variable parameters in the request path.
2. If there are, replace the values with keys to normalize the metrics.
3. Special handling for /{graphs}/ to accommodate multi-graph mode.

[SunnyBoy-WYH]

can we del LOG.debug ?

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 23 lines in your changes missing coverage. Please review.

Project coverage is 1.31%. Comparing base (37e71c4) to head (bf3bfc3).
Report is 11 commits behind head on master.

Files	Patch %	Lines
...g/apache/hugegraph/api/filter/AccessLogFilter.java	0.00%	23 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (37e71c4) and HEAD (bf3bfc3). Click for more details.

HEAD has 4 uploads less than BASE

Flag	BASE (37e71c4)	HEAD (bf3bfc3)
	6	2

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #2569       +/-   ##
============================================
- Coverage     56.42%   1.31%   -55.12%     
+ Complexity      586      30      -556     
============================================
  Files           612     696       +84     
  Lines         49672   55947     +6275     
  Branches       6681    7049      +368     
============================================
- Hits          28029     733    -27296     
- Misses        18833   55161    +36328     
+ Partials       2810      53     -2757     

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

[imbajin]

can we del LOG.debug ?

@SunnyBoy-WYH Thanks, adjust it to trace & use {} format for it, anymore suggestion?