skoppu22 commented on code in PR #334:
URL: https://github.com/apache/cassandra-sidecar/pull/334#discussion_r3059833265


##########
server/src/test/integration/org/apache/cassandra/sidecar/cluster/driver/SidecarLoadBalancingPolicyTest.java:
##########
@@ -42,6 +43,7 @@
 /**
  * A test for the SidecarLoadBalancingPolicy
  */
+@Tag("heavy")

Review Comment:
    1. High node count (5-10) — each in-JVM Cassandra instance holds its own 
heap, memtables,
    caches, etc.
    2. network=true — enables full network simulation in the dtest framework, 
substantially
    increasing memory per node (used by Joining, Moving, Replacement tests)
    3. Dynamic node operations — bootstrap/move/replace require ByteBuddy 
instrumentation and
    maintaining complex state transitions across multiple node lifecycles
   
    When INTEGRATION_MAX_PARALLEL_FORKS=2, multiple 5-10 node tests could run
    simultaneously, easily exceeding the CI container's memory. Moving them to 
the separate
    heavyweight job ensures they run in isolation



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to