HTHou opened a new pull request, #17757:
URL: https://github.com/apache/iotdb/pull/17757
## Description
This PR avoids recording full table-model AST strings in REST table v1
metric labels.
The REST table v1 query and non-query endpoints previously passed
Statement#toString() as the type label to performance_overview, which could
expose long query text such as Query{queryBody=...} and produce
unstable/high-cardinality labels.
The change records stable statement type labels instead:
- query endpoint: QUERY
- table insert through non-query endpoint: INSERT
- other table statements: statement class simple name
I also checked other CommonUtils.addStatementExecutionLatency call sites and
did not find remaining toString() usage for metric type labels.
## Verification
./mvnw -pl external-service-impl/rest -am -DskipTests -DskipITs compile
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
