Skip to content
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

fix some typos #2597

Merged
merged 3 commits into from
May 23, 2022
Merged

fix some typos #2597

merged 3 commits into from
May 23, 2022

Conversation

ming535
Copy link
Contributor

@ming535 ming535 commented May 23, 2022

Which issue does this PR close?

NA

Rationale for this change

fix typo

What changes are included in this PR?

Are there any user-facing changes?

NO

Does this PR break compatibility with Ballista?

NO

@github-actions github-actions bot added the datafusion Changes in the datafusion crate label May 23, 2022
@waynexia
Copy link
Member

There is another identifer in https://github.com/apache/arrow-datafusion/blob/33dcd0273d92936e02237533d4acdef18b4922ba/datafusion/core/src/sql/utils.rs#L474

@xudong963 xudong963 merged commit de5acf9 into apache:master May 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xudong963 xudong963 xudong963 left review comments

@andygrove andygrove andygrove approved these changes

Assignees
No one assigned
Labels
datafusion Changes in the datafusion crate
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ming535 @waynexia @andygrove @xudong963