Skip to content

[Fix-2.1](function) fix wrong nullable signature of function corr - #39380

Merged
yiguolei merged 1 commit into
apache:branch-2.1from
zclllyybb:corr_debug_21
Aug 15, 2024
Merged

[Fix-2.1](function) fix wrong nullable signature of function corr#39380
yiguolei merged 1 commit into
apache:branch-2.1from
zclllyybb:corr_debug_21

Conversation

@zclllyybb

Copy link
Copy Markdown
Contributor

Proposed changes

Issue Number: close #xxx

before corr(nullable_x, nullable_y) will core dump. not fixed.
no need to patch in master because the refactor #37330 already changed the implementation context

@doris-robot

Copy link
Copy Markdown

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

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@zclllyybb

Copy link
Copy Markdown
ContributorAuthor

run buildall

@HappenLeeHappenLee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actionsgithub-actionsBot added the approved Indicates a PR has been approved by one committer. label Aug 14, 2024
@github-actions

Copy link
Copy Markdown
Contributor

PR approved by at least one committer and no changes requested.

@github-actions

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@yiguolei
yiguolei merged commit 4acd695 into apache:branch-2.1Aug 15, 2024
@zclllyybb
zclllyybb deleted the corr_debug_21 branch August 15, 2024 15:24
@zclllyybbzclllyybb changed the title [Fix](function) fix wrong nullable signature of function corr[Fix-2.1](function) fix wrong nullable signature of function corrAug 23, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approvedIndicates a PR has been approved by one committer.reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@zclllyybb@doris-robot@HappenLee@yiguolei