This is an automated email from the ASF dual-hosted git repository.
w41ter pushed a commit to branch branch-2.0
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-2.0 by this push:
new 61aa9cb5d5f [fix](restore) Remove the sqlMode from the view signature
#40615 (#41105)
61aa9cb5d5f is described below
commit 61aa9cb5d5f1fba94c689d36c55aaf9faab3407e
Author: walter <w41te...@gmail.com>
AuthorDate: Mon Sep 23 14:08:00 2024 +0800
[fix](restore) Remove the sqlMode from the view signature #40615 (#41105)
cherry pick from #40615
---
fe/fe-core/src/main/java/org/apache/doris/catalog/View.java | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/fe/fe-core/src/main/java/org/apache/doris/catalog/View.java
b/fe/fe-core/src/main/java/org/apache/doris/catalog/View.java
index 0919ce6c80d..c65297cd864 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/catalog/View.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/catalog/View.java
@@ -223,7 +223,14 @@ public class View extends Table {
sb.append(type);
sb.append(Util.getSchemaSignatureString(fullSchema));
sb.append(inlineViewDef);
- sb.append(sqlMode);
+
+ // ATTN: sqlMode is missing when persist view, so we should not append
it here.
+ //
+ // To keep compatible with the old version, without sqlMode, if the
signature of views
+ // are the same, we think the should has the same sqlMode. (since the
sqlMode doesn't
+ // effect the parsing of inlineViewDef, otherwise the parsing will
fail),
+ //
+ // sb.append(sqlMode);
String md5 = DigestUtils.md5Hex(sb.toString());
LOG.debug("get signature of view {}: {}. signature string: {}", name,
md5, sb.toString());
return md5;
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
