Skip to content

branch-4.0: [fix](array function) Support boolean cast for array first and last #64847#64888

Open
github-actions[bot] wants to merge 1 commit into
branch-4.0from
auto-pick-64847-branch-4.0
Open

branch-4.0: [fix](array function) Support boolean cast for array first and last #64847#64888
github-actions[bot] wants to merge 1 commit into
branch-4.0from
auto-pick-64847-branch-4.0

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Cherry-picked from #64847

…64847)

### What problem does this PR solve?

Problem Summary: `array_first` and `array_last` were rewritten through
`array_filter` before their lambda result type was checked against the
boolean filter argument. When the lambda returned a type that can be
implicitly cast to boolean, analysis reported an error instead of
applying the normal function coercion. This change gives `array_first`
and `array_last` their own signatures so the `array_map` result is
coerced to `array<boolean>` before the functions are rewritten to
`element_at(array_filter(...))`. It also keeps scalar function visitor
dispatch aligned with the new function shape.

### Release note

Fix `array_first` and `array_last` to allow lambda results that can be
implicitly cast to boolean.
@github-actions github-actions Bot requested a review from morningman as a code owner June 26, 2026 08:44
@hello-stephen

Copy link
Copy Markdown
Contributor

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?

@hello-stephen

Copy link
Copy Markdown
Contributor

run buildall

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants