hoist calls for null_sentinel#10356
Conversation
|
run benchmark row_format |
|
🤖 Arrow criterion benchmark running (GKE) | trigger CPU Details (lscpu)Comparing rich-T-kid/optimize-null-array (0018aff) to ee30b61 (merge-base) diff File an issue against this benchmark runner |
|
@Jefffrey I think the benchmarks failed |
|
show benchmark queue |
|
Hi @Jefffrey, you asked to view the benchmark queue (#10356 (comment)).
File an issue against this benchmark runner |
|
🤖 Arrow criterion benchmark completed (GKE) | trigger Instance: CPU Details (lscpu)Details
Resource Usagebase (merge-base)
branch
File an issue against this benchmark runner |
|
thanks @Rich-T-kid |

Which issue does this PR close?
null_sentinel()usage inarrow-row#10246.Rationale for this change
hoist the call to
null_sentnelwhen in a loop since the result wont change.What changes are included in this PR?
hoist the call to
null_sentnelwhen in a loop since the result wont change.Are these changes tested?
yes, existing test
Are there any user-facing changes?
no