teo-tsirpanis opened a new pull request, #393:
URL: https://github.com/apache/arrow-dotnet/pull/393

   ## What's Changed
   
   According to 
[documentation](https://learn.microsoft.com/en-us/dotnet/api/system.gc.addmemorypressure?view=net-10.0#remarks)
 (see also 
https://github.com/dotnet/runtime/issues/33812#issuecomment-601733199):
   
   > The 
[AddMemoryPressure](https://learn.microsoft.com/en-us/dotnet/api/system.gc.addmemorypressure?view=net-10.0)
 and 
[RemoveMemoryPressure](https://learn.microsoft.com/en-us/dotnet/api/system.gc.removememorypressure?view=net-10.0)
 methods improve performance only for types that exclusively depend on 
finalizers to release the unmanaged resources. It's not necessary to use these 
methods in types that follow the dispose pattern, where finalizers are used to 
clean up unmanaged resources only in the event that a consumer of the type 
forgets to call `Dispose`. […]
   
   None of the uses of these methods by Arrow satisfies the above criterion. 
Consequently, this PR removes them, with the exception of 
`MemoryPressureAllocationTracker` whose public API contract is to call these 
methods, and which got obsoleted by this PR.
   
   Closes #50
   Closes #261


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to