[PR] IGNITE-23245 Fixed Placement driver returns assignments only in MS nodes [ignite-3]

2024-11-24 Thread via GitHub
denis-chudov opened a new pull request, #4782: URL: https://github.com/apache/ignite-3/pull/4782 https://issues.apache.org/jira/browse/IGNITE-23245 -- 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

[I] Ignite client nodes are getting terminated when we stop the ignite server. [ignite]

2024-11-24 Thread via GitHub
jinhwan116 opened a new issue, #11680: URL: https://github.com/apache/ignite/issues/11680 ThickClient mode is in use. There is an issue where the client terminates when the ignite server is stopped. version : 2.16.0 Even if the server is stopped, can't the client continue to wait w

Re: [PR] IGNITE-23685 Add local events about the HA zone topology reduce. [ignite-3]

2024-11-24 Thread via GitHub
kgusakov commented on code in PR #4764: URL: https://github.com/apache/ignite-3/pull/4764#discussion_r1855545490 ## modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DistributionZoneManager.java: ## @@ -409,7 +413,15 @@ private void onUpdatePa

Re: [PR] IGNITE-23685 Add local events about the HA zone topology reduce. [ignite-3]

2024-11-24 Thread via GitHub
kgusakov commented on code in PR #4764: URL: https://github.com/apache/ignite-3/pull/4764#discussion_r1855545888 ## modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/events/HaZoneTopologyUpdateEventParams.java: ## @@ -0,0 +1,65 @@ +/* + * Lice

Re: [PR] IGNITE-23685 Add local events about the HA zone topology reduce. [ignite-3]

2024-11-24 Thread via GitHub
kgusakov commented on code in PR #4764: URL: https://github.com/apache/ignite-3/pull/4764#discussion_r1855545748 ## modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/events/HaZoneTopologyUpdateEventParams.java: ## @@ -0,0 +1,65 @@ +/* + * Lice

Re: [PR] IGNITE-23685 Add local events about the HA zone topology reduce. [ignite-3]

2024-11-24 Thread via GitHub
kgusakov commented on code in PR #4764: URL: https://github.com/apache/ignite-3/pull/4764#discussion_r187000 ## modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DistributionZoneManager.java: ## @@ -985,22 +997,23 @@ private CompletableFut

Re: [PR] IGNITE-22557 Create system view for SQL query plans history [ignite]

2024-11-24 Thread via GitHub
oleg-vlsk commented on code in PR #11425: URL: https://github.com/apache/ignite/pull/11425#discussion_r1855373173 ## modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/integration/SqlPlanHistoryIntegrationTest.java: ## @@ -0,0 +1,718 @@ +/* + * Lic

Re: [PR] IGNITE-22557 Create system view for SQL query plans history [ignite]

2024-11-24 Thread via GitHub
oleg-vlsk commented on code in PR #11425: URL: https://github.com/apache/ignite/pull/11425#discussion_r1833674525 ## modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/integration/SqlPlanHistoryIntegrationTest.java: ## @@ -0,0 +1,718 @@ +/* + * Lic

Re: [PR] IGNITE-22557 Create system view for SQL query plans history [ignite]

2024-11-24 Thread via GitHub
oleg-vlsk commented on code in PR #11425: URL: https://github.com/apache/ignite/pull/11425#discussion_r1855666009 ## modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/integration/SqlPlanHistoryIntegrationTest.java: ## @@ -0,0 +1,718 @@ +/* + * Lic

Re: [PR] IGNITE-22557 Create system view for SQL query plans history [ignite]

2024-11-24 Thread via GitHub
oleg-vlsk commented on code in PR #11425: URL: https://github.com/apache/ignite/pull/11425#discussion_r1855373173 ## modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/integration/SqlPlanHistoryIntegrationTest.java: ## @@ -0,0 +1,718 @@ +/* + * Lic

Re: [PR] IGNITE-22557 Create system view for SQL query plans history [ignite]

2024-11-24 Thread via GitHub
oleg-vlsk commented on code in PR #11425: URL: https://github.com/apache/ignite/pull/11425#discussion_r1855670900 ## modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/integration/SqlPlanHistoryIntegrationTest.java: ## @@ -0,0 +1,718 @@ +/* + * Lic

Re: [PR] IGNITE-22557 Create system view for SQL query plans history [ignite]

2024-11-24 Thread via GitHub
oleg-vlsk commented on code in PR #11425: URL: https://github.com/apache/ignite/pull/11425#discussion_r1855666009 ## modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/integration/SqlPlanHistoryIntegrationTest.java: ## @@ -0,0 +1,718 @@ +/* + * Lic

[PR] IGNITE-23758. [ignite-3]

2024-11-24 Thread via GitHub
sanpwc opened a new pull request, #4781: URL: https://github.com/apache/ignite-3/pull/4781 https://issues.apache.org/jira/browse/IGNITE-23758 Not for merge! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

Re: [PR] IGNITE-22557 Create system view for SQL query plans history [ignite]

2024-11-24 Thread via GitHub
oleg-vlsk commented on code in PR #11425: URL: https://github.com/apache/ignite/pull/11425#discussion_r1833672245 ## modules/core/src/main/java/org/apache/ignite/internal/processors/query/running/SqlPlanHistoryTracker.java: ## @@ -0,0 +1,74 @@ +/* + * Licensed to the Apache Soft

Re: [PR] IGNITE-22733 Introduce TransactionChanges [ignite]

2024-11-24 Thread via GitHub
nizhikov commented on code in PR #11675: URL: https://github.com/apache/ignite/pull/11675#discussion_r1855945537 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/transactions/TransactionChanges.java: ## @@ -0,0 +1,83 @@ +/* + * Licensed to the Apache Sof

Re: [PR] IGNITE-22733 Introduce TransactionChanges [ignite]

2024-11-24 Thread via GitHub
nizhikov commented on code in PR #11675: URL: https://github.com/apache/ignite/pull/11675#discussion_r1855946080 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/transactions/TransactionChanges.java: ## @@ -0,0 +1,83 @@ +/* + * Licensed to the Apache Sof

Re: [PR] IGNITE-22733 Introduce TransactionChanges [ignite]

2024-11-24 Thread via GitHub
nizhikov commented on code in PR #11675: URL: https://github.com/apache/ignite/pull/11675#discussion_r1855939036 ## modules/calcite/src/test/java/org/apache/ignite/internal/processors/tx/SqlTransactionsIsolationTest.java: ## @@ -218,32 +96,29 @@ public enum ModifyApi {

Re: [PR] IGNITE-23434 Sql. Provide system view for active locks [ignite-3]

2024-11-24 Thread via GitHub
zstan commented on code in PR #4762: URL: https://github.com/apache/ignite-3/pull/4762#discussion_r1855926622 ## modules/sql-engine/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItLocksSystemViewTest.java: ## @@ -0,0 +1,151 @@ +/* + * Licensed to the Apache Soft

Re: [PR] IGNITE-23754 Do not fail initialized node start because of CMG or MG unavailability [ignite-3]

2024-11-24 Thread via GitHub
rpuch commented on code in PR #4779: URL: https://github.com/apache/ignite-3/pull/4779#discussion_r1856004029 ## modules/raft/src/main/java/org/apache/ignite/internal/raft/RetryContext.java: ## @@ -69,11 +72,19 @@ class RetryContext { * Creates a context. * * @

[PR] IGNITE-23757 .NET: Fix TestDroppedConnectionsAreRestoredInBackground flakiness [ignite-3]

2024-11-24 Thread via GitHub
ptupitsyn opened a new pull request, #4783: URL: https://github.com/apache/ignite-3/pull/4783 (no comment) -- 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

Re: [PR] IGNITE-23733 Sql. Fix flaky test ItSqlQueriesSystemViewTest.multiStatement [ignite-3]

2024-11-24 Thread via GitHub
zstan commented on code in PR #4767: URL: https://github.com/apache/ignite-3/pull/4767#discussion_r1856006523 ## modules/sql-engine/src/integrationTest/java/org/apache/ignite/internal/sql/engine/BaseSqlMultiStatementTest.java: ## @@ -107,6 +115,7 @@ static List> fetchCursors(As

Re: [PR] IGNITE-22557 Create system view for SQL query plans history [ignite]

2024-11-24 Thread via GitHub
oleg-vlsk commented on code in PR #11425: URL: https://github.com/apache/ignite/pull/11425#discussion_r1855874070 ## modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/integration/SqlPlanHistoryIntegrationTest.java: ## @@ -0,0 +1,718 @@ +/* + * Lic

Re: [PR] IGNITE-22557 Create system view for SQL query plans history [ignite]

2024-11-24 Thread via GitHub
oleg-vlsk commented on code in PR #11425: URL: https://github.com/apache/ignite/pull/11425#discussion_r1833674248 ## modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/integration/SqlPlanHistoryIntegrationTest.java: ## @@ -0,0 +1,718 @@ +/* + * Lic

Re: [PR] IGNITE-23623 Fix GridCacheAbstractSelfTest [ignite]

2024-11-24 Thread via GitHub
timoninmaxim merged PR #11666: URL: https://github.com/apache/ignite/pull/11666 -- 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..

Re: [PR] IGNITE-23434 Sql. Provide system view for active locks [ignite-3]

2024-11-24 Thread via GitHub
xtern commented on code in PR #4762: URL: https://github.com/apache/ignite-3/pull/4762#discussion_r1855985640 ## modules/transactions/src/main/java/org/apache/ignite/internal/tx/impl/HeapLockManager.java: ## @@ -251,22 +251,18 @@ public void releaseAll(UUID txId) { }

Re: [PR] IGNITE-23434 Sql. Provide system view for active locks [ignite-3]

2024-11-24 Thread via GitHub
xtern commented on code in PR #4762: URL: https://github.com/apache/ignite-3/pull/4762#discussion_r1855985640 ## modules/transactions/src/main/java/org/apache/ignite/internal/tx/impl/HeapLockManager.java: ## @@ -251,22 +251,18 @@ public void releaseAll(UUID txId) { }

Re: [PR] IGNITE-23434 Sql. Provide system view for active locks [ignite-3]

2024-11-24 Thread via GitHub
xtern commented on code in PR #4762: URL: https://github.com/apache/ignite-3/pull/4762#discussion_r1855988278 ## modules/sql-engine/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItLocksSystemViewTest.java: ## @@ -0,0 +1,151 @@ +/* + * Licensed to the Apache Soft

Re: [PR] IGNITE-23434 Sql. Provide system view for active locks [ignite-3]

2024-11-24 Thread via GitHub
xtern commented on code in PR #4762: URL: https://github.com/apache/ignite-3/pull/4762#discussion_r1855985640 ## modules/transactions/src/main/java/org/apache/ignite/internal/tx/impl/HeapLockManager.java: ## @@ -251,22 +251,18 @@ public void releaseAll(UUID txId) { }

Re: [PR] IGNITE-23754 Do not fail initialized node start because of CMG or MG unavailability [ignite-3]

2024-11-24 Thread via GitHub
ivanzlenko commented on code in PR #4779: URL: https://github.com/apache/ignite-3/pull/4779#discussion_r1855958477 ## modules/raft/src/main/java/org/apache/ignite/internal/raft/RetryContext.java: ## @@ -69,11 +72,19 @@ class RetryContext { * Creates a context. *

Re: [PR] Bump log4j from 2.24.1 to 2.24.2 [ignite-3]

2024-11-24 Thread via GitHub
ptupitsyn merged PR #4780: URL: https://github.com/apache/ignite-3/pull/4780 -- 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...@i