sk0x50 commented on code in PR #5621:
URL: https://github.com/apache/ignite-3/pull/5621#discussion_r2048360591
##########
modules/catalog/src/main/java/org/apache/ignite/internal/catalog/commands/CatalogUtils.java:
##########
@@ -66,6 +67,14 @@ public class CatalogUtils {
*/
public static final String DEFAULT_FILTER = "$..*";
+ /**
+ * Default auto adjust scale up timeout of the default zone.
+ */
+ // In case of enabled colocation the start of each node triggers default
zone rebalance. In order to eliminate such excessive rebalances
+ // default zone auto adjust scale up timeout is set to 5 seconds. If
colocation is disabled tests usually create tables
+ // after all nodes already started meaning that tables are created on
stable topology and usually doesn't assume any rebalances at all.
+ public static final int
DEFAULT_ZONE_DEFAULT_AUTO_ADJUST_SCALE_UP_TIMEOUT_SECONDS = enabledColocation()
? 5 : 0;
Review Comment:
I mean that this variable is `static`, therefore it will be initialized only
once. Let's assume that the test order is:
1. `testPartitionPerTable` (the colocation feature is disabled). So,
`DEFAULT_ZONE_DEFAULT_AUTO_ADJUST_SCALE_UP_TIMEOUT_SECONDS` will be initialized
by `0` (immediate)
2. `testPartitionPerZone` (enabled colocation). This test will use `0` as
well, even though the colocation is enabled.
It seems to me that this behaviour is wrong. Am I missing something?
--
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...@ignite.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
