From cfcc29fb218e7c9f78e1955b50940317b7bd2ae5 Mon Sep 17 00:00:00 2001
From: Mingyu Chen <morningman.cmy@gmail.com>
Date: Wed, 19 Feb 2020 20:19:00 +0800
Subject: [PATCH] [Bug] Missing `in memory` property for old version of
 partition info (#2948)

This bug is introduced by PR #2846
---
 fe/src/main/cup/sql_parser.cup                               | 2 +-
 fe/src/main/java/org/apache/doris/catalog/PartitionInfo.java | 3 +++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/fe/src/main/cup/sql_parser.cup b/fe/src/main/cup/sql_parser.cup
index d6ee476d3e..62f3925a56 100644
--- a/fe/src/main/cup/sql_parser.cup
+++ b/fe/src/main/cup/sql_parser.cup
@@ -1721,7 +1721,7 @@ opt_engine ::=
 
 opt_key_value_map ::=
     {:
-    RESULT = null;
+    RESULT = Maps.newHashMap();
     :}
     | LPAREN key_value_map:map RPAREN
     {:
diff --git a/fe/src/main/java/org/apache/doris/catalog/PartitionInfo.java b/fe/src/main/java/org/apache/doris/catalog/PartitionInfo.java
index 61886a5a00..7aae1c8386 100644
--- a/fe/src/main/java/org/apache/doris/catalog/PartitionInfo.java
+++ b/fe/src/main/java/org/apache/doris/catalog/PartitionInfo.java
@@ -151,6 +151,9 @@ public class PartitionInfo implements Writable {
             idToReplicationNum.put(partitionId, replicationNum);
             if (Catalog.getCurrentCatalogJournalVersion() >= FeMetaVersion.VERSION_72) {
                 idToInMemory.put(partitionId, in.readBoolean());
+            } else {
+                // for compatibility, default is false
+                idToInMemory.put(partitionId, false);
             }
         }
     }