yiguolei commented on code in PR #33383: URL: https://github.com/apache/doris/pull/33383#discussion_r1568522890
########## fe/fe-core/src/main/java/org/apache/doris/qe/Coordinator.java: ########## @@ -4083,6 +4087,54 @@ public List<QueryStatisticsItem.FragmentInstanceInfo> getFragmentInstanceInfos() return result; } + @Override + public void refreshExecStatus() { + List<TNetworkAddress> backendAddresses = Lists.newArrayList(); + if (this.enablePipelineXEngine) { + for (Long backendId : this.beToPipelineExecCtxs.keySet()) { + Backend backend = idToBackend.get(backendId); + backendAddresses.add(new TNetworkAddress(backend.getHost(), backend.getBePort())); + } + } else { + for (Long backendId : this.beToExecStates.keySet()) { + Backend backend = idToBackend.get(backendId); + backendAddresses.add(new TNetworkAddress(backend.getHost(), backend.getBePort())); + } + } + + for (TNetworkAddress address : backendAddresses) { + try { + Client client = ClientPool.backendPool.borrowObject(address); + TGetRealtimeExecStatusRequest req = new TGetRealtimeExecStatusRequest(); + req.setId(this.queryId); + TGetRealtimeExecStatusResponse resp = client.getRealtimeExecStatus(req); + if (!resp.isSetStatus()) { + LOG.warn("Broken GetRealtimeExecStatusResponse response, query {} backend {}", + DebugUtil.printId(queryId), address.toString()); + continue; + } + + if (resp.getStatus().status_code != TStatusCode.OK) { + LOG.warn("Failed to get realtime query exec status, query {} backend {} error msg {}", + DebugUtil.printId(queryId), address.toString(), resp.getStatus().toString()); + continue; + } + + if (!resp.isSetReportExecStatusParams()) { + LOG.warn("Invalid GetRealtimeExecStatusResponse, query {} backend {}", + DebugUtil.printId(queryId), address.toString()); + continue; + } + LOG.info("Get real-time exec status succeed, query {} backend {}", + DebugUtil.printId(queryId), address.toString()); + QeProcessorImpl.INSTANCE.reportExecStatus(resp.getReportExecStatusParams(), address); + } catch (Exception e) { + LOG.warn("Got exception when getRealtimeExecStatus, query {} backend {}", + DebugUtil.printId(queryId), address.toString(), e); + } + } Review Comment: should return object -- 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: commits-unsubscr...@doris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org