[GitHub] [doris] morningman commented on a diff in pull request #22697: [Enhancement](multi-catalog) add a BE selection strategy for hdfs short-circuit-read.

[via GitHub]

morningman commented on code in PR #22697:
URL: https://github.com/apache/doris/pull/22697#discussion_r1287035161


##########
fe/fe-core/src/main/java/org/apache/doris/planner/external/FileQueryScanNode.java:
##########
@@ -326,7 +327,10 @@ public void createScanRangeLocations() throws 
UserException {
             TScanRangeLocation location = new TScanRangeLocation();
             // Use consistent hash to assign the same scan range into the same 
backend among different queries
             Backend selectedBackend = 
ConnectContext.get().getSessionVariable().enableFileCache
-                    ? backendPolicy.getNextConsistentBe(curLocations) : 
backendPolicy.getNextBe();
+                    ? backendPolicy.getNextConsistentBe(curLocations)
+                    : (HdfsResource.enableShortCircuitRead(locationProperties)

Review Comment:
   HdfsResource.enableShortCircuitRead(locationProperties)
   
   this can be called only once



-- 
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