Baymine commented on code in PR #65889:
URL: https://github.com/apache/doris/pull/65889#discussion_r3651771220
##########
fe/fe-core/src/main/java/org/apache/doris/qe/cache/CacheAnalyzer.java:
##########
@@ -459,6 +461,13 @@ private CacheTable
buildCacheTableForOlapScanNode(OlapScanNode node) {
for (Long partitionId : node.getSelectedPartitionIds()) {
Partition partition = olapTable.getPartition(partitionId);
+ if (partition == null) {
Review Comment:
added a catch (RuntimeException e) alongside the existing RpcException catch
around that batch call, so the NPE from a concurrently-dropped partition is
swallowed there and the existing null-check further down throws the controlled
"partition dropped" exception instead.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
