Skip to content

fix a compile error #3502

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
morningman merged 1 commit into from
May 7, 2020
Merged

fix a compile error #3502

morningman merged 1 commit into from
May 7, 2020

Conversation

@yangzhg
Copy link
Member

@yangzhg yangzhg commented May 7, 2020

fix a compile error by no return statment in non-void function

morningman
morningman approved these changes May 7, 2020
View reviewed changes
Copy link
Contributor

@morningman morningman left a comment

Choose a reason for hiding this comment

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

LGTM

@morningman morningman closed this May 7, 2020
@morningman morningman reopened this May 7, 2020
@morningman morningman self-assigned this May 7, 2020
@morningman morningman added area/sql/execution Issues or PRs related to the execution engine kind/fix Categorizes issue or PR as related to a bug. approved Indicates a PR has been approved by one committer. labels May 7, 2020
@morningman morningman merged commit c85d847 into apache:master May 7, 2020
@morningman morningman mentioned this pull request May 7, 2020
Closed
@yangzhg yangzhg deleted the fix_compile_error branch May 8, 2020 02:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@morningman morningman morningman approved these changes

Assignees

@morningman morningman

Labels

approved Indicates a PR has been approved by one committer. area/sql/execution Issues or PRs related to the execution engine kind/fix Categorizes issue or PR as related to a bug.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@yangzhg @morningman