skyyws commented on code in PR #35733:
URL: https://github.com/apache/doris/pull/35733#discussion_r1623711680
##########
fe/fe-core/src/main/java/org/apache/doris/qe/cache/CacheAnalyzer.java:
##########
@@ -496,18 +496,6 @@ private List<CacheTable> buildCacheTableList() {
List<CacheTable> tblTimeList = Lists.newArrayList();
for (int i = 0; i < scanNodes.size(); i++) {
ScanNode node = scanNodes.get(i);
- if (enablePartitionCache()
Review Comment:
按照目前的这个限制，partition cache 功能在任何情况下都不会命中。这个限制，只允许 SQL 中查询一个分区。如果查询大于 1
个分区，则不满足条件，直接返回 CacheMode.None。
但是，如果SQL 中只查询一个分区，最终构造cachePartMap的时候，会由于分区的TooNew，进而导致 map 为空，最终不会将查询结果写到
BE 的缓存，因此每次查询都是 cache miss。
这样的话，分区缓存在任何情况下，都不会起作用。因此，在这个 pr 里面先去掉这个限制，使得分区缓存可以正常起作用。目前暂未发现有数据正确性相关的问题。
--
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
