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

Added in Decimal support to ParquetCachedBatchSerializer #1596

Merged
merged 7 commits into from
Feb 10, 2021

Conversation

razajafri
Copy link
Collaborator

Support added for Decimal to Cache Serializer

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

Signed-off-by: Raza Jafri <rjafri@nvidia.com>
@sameerz sameerz added the task Work required that improves the product but is not user facing label Jan 27, 2021
@sameerz sameerz added this to the Jan 18 - Jan 29 milestone Jan 27, 2021
Signed-off-by: Raza Jafri <rjafri@nvidia.com>
jlowe
jlowe previously approved these changes Jan 27, 2021
@razajafri
Copy link
Collaborator Author

build

@razajafri
Copy link
Collaborator Author

CI is failing against Spark 3.0.0.

I am looking into it and will push an update

Signed-off-by: Raza Jafri <rjafri@nvidia.com>
Signed-off-by: Raza Jafri <rjafri@nvidia.com>
@razajafri
Copy link
Collaborator Author

build

Signed-off-by: Raza Jafri <rjafri@nvidia.com>
@razajafri
Copy link
Collaborator Author

build

Signed-off-by: Raza Jafri <rjafri@nvidia.com>
@razajafri
Copy link
Collaborator Author

build

Signed-off-by: Raza Jafri <rjafri@nvidia.com>
@razajafri
Copy link
Collaborator Author

build

@razajafri
Copy link
Collaborator Author

urm.nvidia.com is unreachable. kicking off again

@razajafri
Copy link
Collaborator Author

build

@razajafri razajafri merged commit d3960e5 into NVIDIA:branch-0.4 Feb 10, 2021
@razajafri razajafri deleted the serializer_dec branch February 10, 2021 22:22
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
* Added in Decimal support to ParquetCachedBatchSerializer

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

* added nested and changed copyrights

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

* fixed test failures

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

* fixed test error

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

* added import

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

* fixed tests for spark311tests profile

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
* Added in Decimal support to ParquetCachedBatchSerializer

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

* added nested and changed copyrights

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

* fixed test failures

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

* fixed test error

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

* added import

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

* fixed tests for spark311tests profile

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