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]