wernerdv commented on code in PR #13540:
URL: https://github.com/apache/ignite/pull/13540#discussion_r3986551036
##########
modules/compatibility/src/test/java/org/apache/ignite/compatibility/testframework/testcontainers/IgniteContainer.java:
##########
@@ -116,18 +107,10 @@ public class IgniteContainer extends
GenericContainer<IgniteContainer> {
/** Base host port for the published thin-client port (node index added).
*/
private static final int CLIENT_HOST_PORT_BASE = 50800;
- /** Custom classes (with their nested classes) used by node in containers.
*/
- private static final List<String> TEST_CLASSES = List.of(
- ContainerAddressResolver.class.getName(),
- TestCompatibilityPluginProvider.class.getName(),
- DisabledRollingUpgradeProcessor.class.getName(),
- DisabledValidationProcessor.class.getName()
- );
-
/** Seconds to wait after SIGTERM before SIGKILL. */
private static final int SHUTDOWN_TIMEOUT_SEC = 30;
- /** Jar holding {@link #TEST_CLASSES}, injected so the old image can load
it. */
+ /** Jar holding the {@link #testClasses() test classes}, injected so the
old image can load it. */
private static volatile File testClassesJar;
Review Comment:
Added caching to ConcurrentHashMap<List, File>.
--
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]
