This is an automated email from the ASF dual-hosted git repository.

yiguolei pushed a commit to branch branch-4.1
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/branch-4.1 by this push:
     new b715c8bbdb9 branch-4.1: [Chore] correct null check in 
`DictionaryManager.dropTableDictionaries()` #63630 (#63964)
b715c8bbdb9 is described below

commit b715c8bbdb93348e28e1cbf110547a006e11631e
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Tue Jun 2 13:53:26 2026 +0800

    branch-4.1: [Chore] correct null check in 
`DictionaryManager.dropTableDictionaries()` #63630 (#63964)
    
    Cherry-picked from #63630
    
    Signed-off-by: Yongtao Huang <[email protected]>
    Co-authored-by: Yongtao Huang <[email protected]>
---
 .../src/main/java/org/apache/doris/dictionary/DictionaryManager.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/dictionary/DictionaryManager.java 
b/fe/fe-core/src/main/java/org/apache/doris/dictionary/DictionaryManager.java
index e503f4045ee..7ebc701c32a 100644
--- 
a/fe/fe-core/src/main/java/org/apache/doris/dictionary/DictionaryManager.java
+++ 
b/fe/fe-core/src/main/java/org/apache/doris/dictionary/DictionaryManager.java
@@ -243,7 +243,7 @@ public class DictionaryManager extends MasterDaemon 
implements Writable {
             Map<String, Long> nameToIds = dictionaryIds.get(dbName);
             for (Long id : dictIds) {
                 Dictionary dict = idToDictionary.remove(id);
-                if (id == null) {
+                if (dict == null) {
                     LOG.warn("Dictionary {} does not exist in dictionaryIds", 
id);
                     continue;
                 }


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to