github-actions[bot] commented on code in PR #60567:
URL: https://github.com/apache/doris/pull/60567#discussion_r3127869549
##########
fe/fe-core/src/main/java/org/apache/doris/httpv2/rest/manager/QueryProfileAction.java:
##########
@@ -480,15 +480,15 @@ public Object currentQueries(HttpServletRequest request,
HttpServletResponse res
LOG.warn("parse query info error: {}", data, e);
}
}
- List<String> titles =
Lists.newArrayList(CurrentQueryStatementsProcNode.TITLE_NAMES);
+ List<String> titles =
Lists.newArrayList(CurrentQueryStatisticsProcDir.TITLE_NAMES);
titles.add(0, FRONTEND);
return ResponseEntityBuilder.ok(new NodeAction.NodeInfo(titles,
queries));
} else {
try {
Review Comment:
This is a separate compatibility break from the already-open
`/current_query_stmts` proc thread. Before this change,
`/rest/v2/manager/query/current_queries` returned the 8-column statement view
from `CurrentQueryStatementsProcNode`; switching this endpoint to
`CurrentQueryStatisticsProcDir` changes both schema and semantics for every
existing REST caller. Even if the proc entries are restored, this manager API
still needs to keep the old node (or move the richer statistics table to a
new/versioned endpoint) to avoid breaking clients.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
