mhansonp commented on PR #7608: URL: https://github.com/apache/geode/pull/7608#issuecomment-1111867194
So there is one remaining problem in the stress test and that is caused by the fact that the test DurableClientSimpleDUnitTest.testReadyForEventsNotCalledImplicitlyForRegisterInterestWithCacheXML uses a server with a defined port number of 10188. So when the test starts up it references an cache.xml that has the port defined in it. This means that the test cannot be run in parallel with itself as is done through the stress test and be ok. Depending on timing it will have resource contention. This has been in there for quite some time and is not really a problem except in stress test. If we do want to fix it, we will need to find a way to override it by generating it on the fly or modifying the XML on the fly. Generating it on the fly would be the easier path. -- 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: notifications-unsubscr...@geode.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
