-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Make pyarrow dependency optional for tests #2733
Conversation
A documentation preview will be available soon. Request a new doc build by commenting
If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here. |
buildkite test this please |
b98b45b
to
733d5e2
Compare
I've just fixed the lint issues. Sorry, I override your commit fixing the lint, we did it at the same time and I just amend. |
buildkite test this please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! LGTM.
I fixed the lint with |
buildkite test this please |
(cherry picked from commit d692c70)
(cherry picked from commit d692c70)
Thank you for your contribution! And sorry for pushing to your branch. |
This change allows to run tests without having pyarrow installed.