-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
CLN: consolidate compression inference as much as possible #12688
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
Comments
cc @lababidi |
@gfyoung if you are looking for something to do :> I think we can blow away there is also compression handling in xref #13317 |
Haha. Thanks @jreback for the mention. I'll take a closer look when I have the time. 😄 |
@jreback I got the following error when attempting to test:
|
you need to rebuild extensions
|
Part of pandas-dev#14576. Closes pandas-dev#12688. Closes pandas-dev#14570. Opens pandas-dev#14874.
Part of pandas-dev#14576. Closes pandas-dev#12688. Closes pandas-dev#14570. Opens pandas-dev#14874.
closes pandas-dev#14576 closes pandas-dev#12688 closes pandas-dev#14570 xref pandas-dev#14874
we have 3 copies of the compression inference routines in
parser.py, parser.py, and io/common.py
. See how much we can reduce this (some is necessary as sometimes we need to actually return a file handle or need to read in and re-wrap).xref #12175
The text was updated successfully, but these errors were encountered: