lhotari opened a new pull request, #21328:
URL: https://github.com/apache/pulsar/pull/21328
Fixes #21291
Fixes #21285
### Motivation
NarUnpackerTest is really flaky. See the above isues that this PR fixes.
When investigating the problem, I was able to reproduce the issue by running
the test within Docker with constrained resources. There's a 30 second timeout
in the shouldExtractFilesOnceInDifferentProcess test for the 10 different
processes to complete. This takes much longer since each process uses
significant amount of CPU. The fix is to reduce the CPU consumption by reducing
the test data size, reducing the number of processes and tuning the JVM
parameters for the forked processes.
### Modifications
- reduce number of processes from 10 to 5
- reduce the number of files in the sample zip from 10000 to 5000
- make the test use less CPU resources so that it will run in CI with
constrained resources
- make JIT compiler stop at C1
- disable JMX registration in Log4j2 which showed up in profiling
- increase max heap size from 64MB to 96MB to give more head room for GC
### Documentation
<!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
- [ ] `doc` <!-- Your PR contains doc changes. -->
- [ ] `doc-required` <!-- Your PR changes impact docs and you will update
later -->
- [x] `doc-not-needed` <!-- Your PR changes do not impact docs -->
- [ ] `doc-complete` <!-- Docs have been already added -->
--
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]
