[GitHub] [doris] eldenmoon commented on a diff in pull request #21573: [Bug](point query) cancel future when meet timeout in PointQueryExec

["eldenmoon (via GitHub)" <gi...@apache.org>]

eldenmoon commented on code in PR #21573:
URL: https://github.com/apache/doris/pull/21573#discussion_r1257466207


##########
fe/fe-core/src/main/java/org/apache/doris/qe/PointQueryExec.java:
##########
@@ -185,31 +185,36 @@ private RowBatch getNextInternal(Status status, Backend backend) throws TExcepti
             while (pResult == null) {
                 InternalService.PTabletKeyLookupRequest request = requestBuilder.build();
                 Future<InternalService.PTabletKeyLookupResponse> futureResponse =
-                        BackendServiceProxy.getInstance().fetchTabletDataAsync(backend.getBrpcAdress(), request);
+                         BackendServiceProxy.getInstance().fetchTabletDataAsync(backend.getBrpcAdress(), request);
                 long currentTs = System.currentTimeMillis();
                 if (currentTs >= timeoutTs) {
                     LOG.warn("fetch result timeout {}", backend.getBrpcAdress());
                     status.setStatus("query timeout");
                     return null;
                 }
                 try {
-                    pResult = futureResponse.get(timeoutTs - currentTs, TimeUnit.MILLISECONDS);
+                    pResult = futureResponse.get((timeoutTs - currentTs) / 3, TimeUnit.MILLISECONDS);
                 } catch (InterruptedException e) {

Review Comment:
   modified



-- 
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-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org