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

[WIP] Proper index handling for chunked reading multiple parquet files #8726

Closed
wants to merge 1 commit into from

Conversation

galipremsagar
Copy link
Contributor

@galipremsagar galipremsagar commented Jul 13, 2021

Fixes: #8536, #6872
This pr is WIP

@github-actions github-actions bot added the Python Affects Python cuDF API. label Jul 13, 2021
@codecov
Copy link

codecov bot commented Jul 13, 2021

Codecov Report

❗ No coverage uploaded for pull request base (branch-21.08@7823a18). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head 0c6cb83 differs from pull request most recent head 12c2755. Consider uploading reports for the commit 12c2755 to get more accurate results
Impacted file tree graph

@@               Coverage Diff               @@
##             branch-21.08    #8726   +/-   ##
===============================================
  Coverage                ?   10.66%           
===============================================
  Files                   ?      109           
  Lines                   ?    18692           
  Branches                ?        0           
===============================================
  Hits                    ?     1993           
  Misses                  ?    16699           
  Partials                ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7823a18...12c2755. Read the comment docs.

@galipremsagar
Copy link
Contributor Author

galipremsagar commented Jul 19, 2021

Moving this to 21.10 as this is a major change at the python layer that I'm not comfortable merging at the last moment.

cc: @vuule @devavret @quasiben @JohnZed

@galipremsagar galipremsagar changed the base branch from branch-21.08 to branch-21.10 July 19, 2021 22:28
@github-actions
Copy link

This PR has been labeled inactive-90d due to no recent activity in the past 90 days. Please close this PR if it is no longer required. Otherwise, please respond with a comment indicating any updates.

@galipremsagar
Copy link
Contributor Author

duplicate of #11105

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Python Affects Python cuDF API.
Projects
None yet
1 participant