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.

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

chore: Fix incorrect comment in substrait consumer #9123

Merged
merged 1 commit into from
Feb 5, 2024

Conversation

caicancai
Copy link
Member

@caicancai caicancai commented Feb 4, 2024

Which issue does this PR close?

Fix incorrect comment in substrait consumer

Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@caicancai
Copy link
Member Author

caicancai commented Feb 4, 2024

If my pr is wrong, you can close directly

@andygrove andygrove changed the title chore:rename DataFrame to LogicalPlan chore: Fix incorrect comment in substrait consumer Feb 4, 2024
@alamb alamb merged commit aef28ae into apache:main Feb 5, 2024
25 checks passed
@alamb alamb added the documentation Improvements or additions to documentation label Feb 5, 2024
@alamb
Copy link
Contributor

alamb commented Feb 5, 2024

Thanks @caicancai

@caicancai caicancai deleted the rename branch February 5, 2024 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation substrait
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants