-
-
Notifications
You must be signed in to change notification settings - Fork 18.1k
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
Fixes mypy attribute issue in io/parquet by adding a hasattr check #45570
Fixes mypy attribute issue in io/parquet by adding a hasattr check #45570
Conversation
Anirudhsekar96
commented
Jan 23, 2022
- closes closes #44914 by changing the path object to string if it is of io.BufferWriter #45480, BUG: Partial file write to disk on calling to_parquet() with engine='pyarrow' with unsupported dtype #44914
- Passes tests in pytest
- Ensure all linting tests pass, see here for how to run them
Instead of checking for Probably would also make sense to change the if condition to |
@Anirudhsekar96 great ping on green |
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.
One comment otherwise LGTM
great ping on green |
The checks have completed running |
Thank you @Anirudhsekar96 |
…andas-dev#45570) * Fixes mypy attribute issue in io/parquet by adding a hasattr check * changed attr to name attribute inside raw * combined the conditional checks on io.BufferedWriter in io/parquet * changed attribute from raw.name to name and added hasattr check * Skipped checks for graceful cleanup in windows * Changed checks for path object in parquet, linted and sorted imports * Removed sys dependency and changed pytest check for windows * Changed the condition in io/parquet to hasattr from isinstance of io.FileIO * Changed the pytest skip condition to decorator and added a message * Changed pytest skipif to mark.xfail