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

Removed StructTypes support for ParquetCachedBatchSerializer as cudf doesn't support it yet #1723

Merged
merged 2 commits into from
Feb 13, 2021

Conversation

razajafri
Copy link
Collaborator

I whitelisted StructTypes thinking they are supported or very close to being supported in cudf. This won't be possible until rapidsai/7230 is merged

Signed-off-by: Raza Jafri rjafri@nvidia.com

Signed-off-by: Raza Jafri <rjafri@nvidia.com>
@razajafri razajafri changed the title Removed whitelisting of StructTypes as cudf doesn't support it Removed StructTypes support for ParquetCachedBatchSerializer as cudf doesn't support it yet Feb 12, 2021
Signed-off-by: Raza Jafri <rjafri@nvidia.com>
@razajafri
Copy link
Collaborator Author

build

@sameerz sameerz added the task Work required that improves the product but is not user facing label Feb 13, 2021
@sameerz sameerz added this to the Feb 1 - Feb 12 milestone Feb 13, 2021
@razajafri razajafri merged commit da6b892 into NVIDIA:branch-0.4 Feb 13, 2021
@razajafri razajafri deleted the remove_struct_whitelist branch February 16, 2021 16:57
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
…doesn't support it yet (NVIDIA#1723)

* Removed whitelisting of StructTypes as cudf doesn't support it

Signed-off-by: Raza Jafri <rjafri@nvidia.com>

* removed nested keyword

Signed-off-by: Raza Jafri <rjafri@nvidia.com>

Co-authored-by: Raza Jafri <rjafri@nvidia.com>
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
…doesn't support it yet (NVIDIA#1723)

* Removed whitelisting of StructTypes as cudf doesn't support it

Signed-off-by: Raza Jafri <rjafri@nvidia.com>

* removed nested keyword

Signed-off-by: Raza Jafri <rjafri@nvidia.com>

Co-authored-by: Raza Jafri <rjafri@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
task Work required that improves the product but is not user facing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants