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

Closed unclosed views to avoid memory leak #2392

Merged
merged 2 commits into from
May 11, 2021
Merged

Conversation

razajafri
Copy link
Collaborator

fixes #2355

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

Signed-off-by: Raza Jafri <rjafri@nvidia.com>
Signed-off-by: Raza Jafri <rjafri@nvidia.com>
@razajafri razajafri requested a review from jlowe May 11, 2021 19:50
@jlowe
Copy link
Member

jlowe commented May 11, 2021

build

@jlowe jlowe merged commit 16045f3 into NVIDIA:branch-0.6 May 11, 2021
@sameerz sameerz added the bug Something isn't working label May 12, 2021
@sameerz sameerz added this to the May 10 - May 21 milestone May 12, 2021
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
* Closed unclosed views to avoid memory leak

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

* addressed review comments

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
* Closed unclosed views to avoid memory leak

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

* addressed review comments

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
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] convertDecimal64ToDecimal32Wrapper leaks ColumnView instances
3 participants