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

morningman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-doris.git


The following commit(s) were added to refs/heads/master by this push:
     new c98b411  [Bug] Revert part of #4199 to avoid BE crash(#4269)
c98b411 is described below

commit c98b4115003f370730ee10c83bf4ce1aa26b7d90
Author: EmmyMiao87 <[email protected]>
AuthorDate: Thu Aug 6 19:06:00 2020 +0800

    [Bug] Revert part of #4199 to avoid BE crash(#4269)
    
    Revert “Change type of sum, min, max function column in mv”
    
    This pr is revert pr #4199 .
    The daily test is cored when the type of mv column has been changed.
    So I revert the pr.
    The daily core will be fixed in the future. After that, the pr#4199 will be 
enable.
    
    Change-Id: Ie04fcfacfcd38480121addc5e454093d4ae75181
---
 fe/fe-core/src/main/java/org/apache/doris/analysis/MVColumnItem.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/analysis/MVColumnItem.java 
b/fe/fe-core/src/main/java/org/apache/doris/analysis/MVColumnItem.java
index 37137e4..523ab09 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/analysis/MVColumnItem.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/analysis/MVColumnItem.java
@@ -105,7 +105,9 @@ public class MVColumnItem {
         } else {
             Column result = new Column(baseColumn);
             result.setIsKey(isKey);
-            result.setType(type);
+            // If the mv column type is inconsistent with the base column 
type, the daily test will core.
+            // So, I comment this line firstly.
+//            result.setType(type);
             result.setAggregationType(aggregationType, 
isAggregationTypeImplicit);
             return result;
         }


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

Reply via email to