Michael Smith has posted comments on this change. ( http://gerrit.cloudera.org:8080/24997 )
Change subject: IMPALA-11917: Codegen branch-free reads of nullable scalar slots ...................................................................... Patch Set 9: (2 comments) http://gerrit.cloudera.org:8080/#/c/24997/7/be/src/exprs/slot-ref.cc File be/src/exprs/slot-ref.cc: http://gerrit.cloudera.org:8080/#/c/24997/7/be/src/exprs/slot-ref.cc@239 PS7, Line 239: // Only types whose value is a single scalar load are supported; others have multiple > I understand struct/collection, but what is the reason for excluding the ot Added some comments. Simple scalar types showed a benefit, others don't and require additional code to support. http://gerrit.cloudera.org:8080/#/c/24997/7/be/src/exprs/slot-ref.cc@278 PS7, Line 278: is_null, llvm::Constant::getNullValue(val->getType()), val, "val_or_zero"); : : return CodegenAnyVal::GetN > Create a CodegenAnyVal functions for this, e.g. getNullableVal()? Done -- To view, visit http://gerrit.cloudera.org:8080/24997 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I6edaf90100cc3360a7c93d3218eb6956fcec24f0 Gerrit-Change-Number: 24997 Gerrit-PatchSet: 9 Gerrit-Owner: Michael Smith <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]> Gerrit-Comment-Date: Tue, 06 Oct 2026 16:33:44 +0000 Gerrit-HasComments: Yes
