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