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.

Sign up for GitHub

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

Jump to bottom

[Java] Call allocation listener on BaseAllocator#wrapForeignAllocation #34230

Closed
zhztheplayer opened this issue Feb 17, 2023 · 0 comments · Fixed by #34231
Closed

[Java] Call allocation listener on BaseAllocator#wrapForeignAllocation #34230

zhztheplayer opened this issue Feb 17, 2023 · 0 comments · Fixed by #34231
Assignees
@zhztheplayer
Labels
Component: Java Type: bug
Milestone
12.0.0

Comments

@zhztheplayer
Copy link
Member

Describe the bug, including details regarding any error messages, version, and platform.

Buffer created by BaseAllocator#wrapForeignAllocation calls AllocationListener#onRelease but the Allocator didn't call AllocationListener#onAllocate() when the buffer was created.

Component(s)

Java
@zhztheplayer zhztheplayer mentioned this issue Feb 17, 2023
Merged
@lidavidm lidavidm added this to the 12.0.0 milestone Feb 21, 2023
lidavidm pushed a commit that referenced this issue Feb 21, 2023
@zhztheplayer
GH-34230: [Java] Call allocation listener on BaseAllocator#wrapForeig…
6850923 
…nAllocation (#34231)

#34230 
* Closes: #34230

Lead-authored-by: Hongze Zhang <hongze.zhang@intel.com>
Co-authored-by: Hongze Zhang <mailtozhz@126.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
fatemehp pushed a commit to fatemehp/arrow that referenced this issue Feb 24, 2023
@zhztheplayer @fatemehp
apacheGH-34230: [Java] Call allocation listener on BaseAllocator#wrap…
8afecfa 
…ForeignAllocation (apache#34231)

apache#34230 
* Closes: apache#34230

Lead-authored-by: Hongze Zhang <hongze.zhang@intel.com>
Co-authored-by: Hongze Zhang <mailtozhz@126.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zhztheplayer zhztheplayer

Labels
Component: Java Type: bug
Projects
None yet
Milestone
12.0.0
2 participants
@lidavidm @zhztheplayer