mhansonp commented on pull request #6748: URL: https://github.com/apache/geode/pull/6748#issuecomment-896152905
> As for altering the test framework, my assessment (for now) is that including these few tests in stress test runs isn't worth the extra complexity in the test framework. I suggested that we modify this test to be repeated test capable rather than adding a new category which you have done. E.g. Pulling the default port in into a default port test. Then modify this code to test that it works fine with a configured port. I am not very attached to this idea, but it seems cleaner. Because then the redis code will get repeat testing. -- 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]
