Skip to content

Commit

Permalink
[BugFix] dictionary_get did not support third params for materialized…
Browse files Browse the repository at this point in the history
… view (backport #52125) (#52222)

Co-authored-by: Rohit Satardekar <rohitrs1983@gmail.com>
  • Loading branch information
mergify[bot] and rohitrs1983 authored Oct 23, 2024
1 parent 4d35762 commit 5ac1d7f
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,8 @@ protected DictionaryGetExpr(DictionaryGetExpr other) {
@Override
protected String toSqlImpl() {
String message = "DICTIONARY_GET(";
for (int i = 0; i < this.children.size(); ++i) {
int size = (this.children.size() == 3) ? this.children.size() - 1 : this.children.size();
for (int i = 0; i < size; ++i) {
Expr expr = this.children.get(i);
message += expr.toSql();
message += ", ";
Expand Down

0 comments on commit 5ac1d7f

Please sign in to comment.