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

GH-33649: [C++] Fix documentation on Newly Added Methods on ExecPlan #33651

Closed
wants to merge 2 commits into from

Conversation

vibhatha
Copy link
Collaborator

@vibhatha vibhatha commented Jan 13, 2023

This is a minor fix for CI failure in #33623 (comment).

TODO

  • Self review
  • Check for CI status
  • Mark PR ready for review

@vibhatha vibhatha marked this pull request as ready for review January 13, 2023 06:55
@github-actions
Copy link

@vibhatha vibhatha marked this pull request as ready for review January 13, 2023 16:56
@westonpace
Copy link
Member

Ah, sorry, I did not see this when I logged on this morning and I fixed this on my existing docs PR.

2213698

Can you review my wording? If it's ok we can probably close this.

@vibhatha
Copy link
Collaborator Author

closing this PR in favor of 2213698

@vibhatha vibhatha closed this Jan 14, 2023
@vibhatha
Copy link
Collaborator Author

Ah, sorry, I did not see this when I logged on this morning and I fixed this on my existing docs PR.

2213698

Can you review my wording? If it's ok we can probably close this.

Sorry I wasn't online at that time. And I see the PR is merged, the wording is fine. Since, I am closing this one.

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

Successfully merging this pull request may close these issues.

[C++] Fix documentation on Newly Added Methods on ExecPlan
2 participants