venkata91 commented on PR #26592:
URL: https://github.com/apache/flink/pull/26592#issuecomment-2906065317

   > Thanks for the patch. It is a good catch.
   > 
   > I understand it might be a little tricky to do unit test. but we may be 
able to do integration test. We can do the following:
   > 
   > 1. start a session cluster with normal size JM/TM ( < 32GB )
   > 2. when launch the job on the command line, use an additional JVM arg of 
`-XX:-UseCompressedOops` to force a 24-bytes object header with JVM memory > 4 
GB.
   > 
   > This way we should be able to test it.
   
   Thanks for reviewing the patch !
   
   Yes, I'm working on e-2-e test with `FlinkContainers`. In this case, we need 
2 different JVMs (JM and TM) one with `UseCompressedOops` and one without that. 
Still working on the e-2-e test, will post it once it is ready.


-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to