-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[C++][Parquet] Not all decoders are passed a memory pool from column reader #39525
Comments
take |
mapleFU
pushed a commit
that referenced
this issue
Jan 11, 2024
### Rationale for this change Memory pools should be plumbed through where ever possible. ### What changes are included in this PR? Pass through memory pool to decoders ### Are these changes tested? Not directly; this was caught via some internal fuzz targets. ### Are there any user-facing changes? No. * Closes: #39525 Authored-by: Micah Kornfield <micahk@google.com> Signed-off-by: mwish <maplewish117@gmail.com>
clayburn
pushed a commit
to clayburn/arrow
that referenced
this issue
Jan 23, 2024
…9526) ### Rationale for this change Memory pools should be plumbed through where ever possible. ### What changes are included in this PR? Pass through memory pool to decoders ### Are these changes tested? Not directly; this was caught via some internal fuzz targets. ### Are there any user-facing changes? No. * Closes: apache#39525 Authored-by: Micah Kornfield <micahk@google.com> Signed-off-by: mwish <maplewish117@gmail.com>
dgreiss
pushed a commit
to dgreiss/arrow
that referenced
this issue
Feb 19, 2024
…9526) ### Rationale for this change Memory pools should be plumbed through where ever possible. ### What changes are included in this PR? Pass through memory pool to decoders ### Are these changes tested? Not directly; this was caught via some internal fuzz targets. ### Are there any user-facing changes? No. * Closes: apache#39525 Authored-by: Micah Kornfield <micahk@google.com> Signed-off-by: mwish <maplewish117@gmail.com>
raulcd
pushed a commit
that referenced
this issue
Feb 20, 2024
### Rationale for this change Memory pools should be plumbed through where ever possible. ### What changes are included in this PR? Pass through memory pool to decoders ### Are these changes tested? Not directly; this was caught via some internal fuzz targets. ### Are there any user-facing changes? No. * Closes: #39525 Authored-by: Micah Kornfield <micahk@google.com> Signed-off-by: mwish <maplewish117@gmail.com>
zanmato1984
pushed a commit
to zanmato1984/arrow
that referenced
this issue
Feb 28, 2024
…9526) ### Rationale for this change Memory pools should be plumbed through where ever possible. ### What changes are included in this PR? Pass through memory pool to decoders ### Are these changes tested? Not directly; this was caught via some internal fuzz targets. ### Are there any user-facing changes? No. * Closes: apache#39525 Authored-by: Micah Kornfield <micahk@google.com> Signed-off-by: mwish <maplewish117@gmail.com>
thisisnic
pushed a commit
to thisisnic/arrow
that referenced
this issue
Mar 8, 2024
…9526) ### Rationale for this change Memory pools should be plumbed through where ever possible. ### What changes are included in this PR? Pass through memory pool to decoders ### Are these changes tested? Not directly; this was caught via some internal fuzz targets. ### Are there any user-facing changes? No. * Closes: apache#39525 Authored-by: Micah Kornfield <micahk@google.com> Signed-off-by: mwish <maplewish117@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug, including details regarding any error messages, version, and platform.
This can prevent proper memory attribution/etc from occurring.
Component(s)
C++
The text was updated successfully, but these errors were encountered: