Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/24557 )
Change subject: WIP: IMPALA-15057: Add variant_get() builtin and first-class VARIANT expressions ...................................................................... Patch Set 3: (7 comments) gerrit-auto-critic failed. You can reproduce it locally using command: python3 bin/jenkins/critique-gerrit-review.py --dryrun To run it, you might need a virtual env with Python3's venv installed. http://gerrit.cloudera.org:8080/#/c/24557/3/be/src/exprs/anyval-util.h File be/src/exprs/anyval-util.h: http://gerrit.cloudera.org:8080/#/c/24557/3/be/src/exprs/anyval-util.h@350 PS3, Line 350: // slot -> VariantVal conversion already happened in SlotRef::GetVariantValInterpreted. line too long (95 > 90) http://gerrit.cloudera.org:8080/#/c/24557/3/be/src/exprs/scalar-expr-evaluator.cc File be/src/exprs/scalar-expr-evaluator.cc: http://gerrit.cloudera.org:8080/#/c/24557/3/be/src/exprs/scalar-expr-evaluator.cc@387 PS3, Line 387: // and value). Store it in result_ and return a pointer to it, mirroring TYPE_STRUCT. line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/24557/3/be/src/exprs/variant-functions-ir.cc File be/src/exprs/variant-functions-ir.cc: http://gerrit.cloudera.org:8080/#/c/24557/3/be/src/exprs/variant-functions-ir.cc@129 PS3, Line 129: int8_t t; if (!sub.GetInt8(&t)) return Coerce::MISMATCH; *out = t; return Coerce::OK; line too long (93 > 90) http://gerrit.cloudera.org:8080/#/c/24557/3/be/src/exprs/variant-functions-ir.cc@174 PS3, Line 174: int8_t t; if (!sub.GetInt8(&t)) return Coerce::MISMATCH; *out = t; return Coerce::OK; line too long (93 > 90) http://gerrit.cloudera.org:8080/#/c/24557/3/be/src/exprs/variant-functions-ir.cc@320 PS3, Line 320: // DATE stores days-since-epoch; INT32 is accepted as a day count (see coercion table). line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/24557/3/be/src/util/variant-util.cc File be/src/util/variant-util.cc: http://gerrit.cloudera.org:8080/#/c/24557/3/be/src/util/variant-util.cc@64 PS3, Line 64: return VariantToJson(v.metadata.ptr, v.metadata.len, v.value.ptr, v.value.len, json_out); line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/24557/3/fe/src/main/java/org/apache/impala/analysis/FunctionCallExpr.java File fe/src/main/java/org/apache/impala/analysis/FunctionCallExpr.java: http://gerrit.cloudera.org:8080/#/c/24557/3/fe/src/main/java/org/apache/impala/analysis/FunctionCallExpr.java@763 PS3, Line 763: throw new AnalysisException(fnName_.getFunction() + "() unsupported target type '" line too long (92 > 90) -- To view, visit http://gerrit.cloudera.org:8080/24557 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I55bfed394ac2fb57135fadedd489f59e4cc10de4 Gerrit-Change-Number: 24557 Gerrit-PatchSet: 3 Gerrit-Owner: Zoltan Borok-Nagy <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Comment-Date: Mon, 03 Aug 2026 16:07:39 +0000 Gerrit-HasComments: Yes
