This is an automated email from the ASF dual-hosted git repository.
eldenmoon pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-3.0 by this push:
new 54dd4a3ed7a [pick](short circuit) avoid set cacheId when non prepared
execute (#40910)
54dd4a3ed7a is described below
commit 54dd4a3ed7a03a6d9598ff5dcfa657344c6764bc
Author: lihangyu <[email protected]>
AuthorDate: Fri Sep 20 10:54:11 2024 +0800
[pick](short circuit) avoid set cacheId when non prepared execute (#40910)
#40525
---
fe/fe-core/src/main/java/org/apache/doris/qe/PointQueryExecutor.java | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git
a/fe/fe-core/src/main/java/org/apache/doris/qe/PointQueryExecutor.java
b/fe/fe-core/src/main/java/org/apache/doris/qe/PointQueryExecutor.java
index dae9a94db39..9e4030b768b 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/qe/PointQueryExecutor.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/qe/PointQueryExecutor.java
@@ -267,7 +267,10 @@ public class PointQueryExecutor implements CoordInterface {
if (snapshotVisibleVersions != null &&
!snapshotVisibleVersions.isEmpty()) {
requestBuilder.setVersion(snapshotVisibleVersions.get(0));
}
- if (shortCircuitQueryContext.cacheID != null) {
+ // Only set cacheID for prepared statement excute phase,
+ // otherwise leading to many redundant cost in BE side
+ if (shortCircuitQueryContext.cacheID != null
+ && ConnectContext.get().command ==
MysqlCommand.COM_STMT_EXECUTE) {
InternalService.UUID.Builder uuidBuilder =
InternalService.UUID.newBuilder();
uuidBuilder.setUuidHigh(shortCircuitQueryContext.cacheID.getMostSignificantBits());
uuidBuilder.setUuidLow(shortCircuitQueryContext.cacheID.getLeastSignificantBits());
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
