zhztheplayer opened a new pull request, #9321:
URL: https://github.com/apache/incubator-gluten/pull/9321

   Previously the feature uses `total memory - free memory` as the on-heap 
occupied size, which was wrong. JVM doesn't return the free memory back to 
operating system so the free memory is only a concept inside JVM internal. The 
heap occupation from operating system is always the total JVM memory.
   
   The patch correct the calculation by using the total JVM memory instead.
   
   Related:
   
   https://github.com/apache/incubator-gluten/pull/5439
   https://github.com/apache/incubator-gluten/issues/9276
   https://github.com/apache/incubator-gluten/discussions/8600
   


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to