[refine](expr) Use the new execute interface of expr in some places. … #59879
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
…(#59315)
#59315
before
Block block; block.insert({ColumnUInt8::create(1), std::make_shared<DataTypeUInt8>(), ""}); int result = -1; RETURN_IF_ERROR(ctx->execute(&block, &result)); DCHECK(result != -1); auto column = block.get_by_position(result).column; DCHECK(column->size() == 1);now
ColumnWithTypeAndName result; RETURN_IF_ERROR(ctx->execute_const_expr(result)); DCHECK(result.column->size() == 1);What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)