Re: [PR] IGNITE-23414 : Calcite. Fix cast of dynamic parameters [ignite]

2024-12-29 Thread via GitHub
Vladsz83 closed pull request #11621: IGNITE-23414 : Calcite. Fix cast of dynamic parameters URL: https://github.com/apache/ignite/pull/11621 -- 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 spec

Re: [PR] IGNITE-23414 : Calcite. Fix cast of dynamic parameters [ignite]

2024-12-29 Thread via GitHub
Vladsz83 commented on PR #11621: URL: https://github.com/apache/ignite/pull/11621#issuecomment-2564766412 Impl. in #11787 -- 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

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

2024-12-29 Thread via GitHub
karthikrsubs commented on issue #11680: URL: https://github.com/apache/ignite/issues/11680#issuecomment-2564717004 We are also observing the similar errors in Server Node too . It would be great if some apache experts solve this issue -- This is an automated message from the Apache Git Se

Re: [PR] IGNITE-23870 Implement a method to receive a chain of topology changes [ignite-3]

2024-12-29 Thread via GitHub
sanpwc commented on code in PR #4966: URL: https://github.com/apache/ignite-3/pull/4966#discussion_r1899119228 ## modules/table/src/integrationTest/java/org/apache/ignite/internal/disaster/ItDisasterRecoveryReconfigurationTest.java: ## @@ -1234,6 +1242,295 @@ void testTwoPhaseRe

Re: [PR] IGNITE-23748 Lock LWM when executing RO operation on data node [ignite-3]

2024-12-29 Thread via GitHub
rpuch commented on PR #4974: URL: https://github.com/apache/ignite-3/pull/4974#issuecomment-2565093203 > Do you have benchmark results that will show the difference for RO performance with and without proposed logic? I added information in benchmark results to the PR description. In s

Re: [PR] IGNITE-23304 [ignite-3]

2024-12-29 Thread via GitHub
sanpwc commented on code in PR #4821: URL: https://github.com/apache/ignite-3/pull/4821#discussion_r1899327370 ## modules/raft/src/main/java/org/apache/ignite/internal/raft/server/impl/JraftServerImpl.java: ## @@ -447,6 +447,10 @@ public boolean startRaftNode( // Th

Re: [PR] IGNITE-23304 [ignite-3]

2024-12-29 Thread via GitHub
sanpwc commented on code in PR #4821: URL: https://github.com/apache/ignite-3/pull/4821#discussion_r1899325767 ## modules/raft-api/src/main/java/org/apache/ignite/internal/raft/Command.java: ## @@ -17,10 +17,27 @@ package org.apache.ignite.internal.raft; +import org.apache.

Re: [PR] IGNITE-23304 [ignite-3]

2024-12-29 Thread via GitHub
sanpwc commented on code in PR #4821: URL: https://github.com/apache/ignite-3/pull/4821#discussion_r1899328977 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/raft/PartitionListener.java: ## @@ -241,30 +215,46 @@ private void onWriteBusy(Iterator> it

Re: [PR] IGNITE-23304 [ignite-3]

2024-12-29 Thread via GitHub
sanpwc commented on code in PR #4821: URL: https://github.com/apache/ignite-3/pull/4821#discussion_r1899328977 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/raft/PartitionListener.java: ## @@ -241,30 +215,46 @@ private void onWriteBusy(Iterator> it

Re: [PR] IGNITE-24025 Introduce catalog command to drop schema [ignite-3]

2024-12-29 Thread via GitHub
xtern commented on code in PR #4972: URL: https://github.com/apache/ignite-3/pull/4972#discussion_r1899341403 ## modules/catalog/src/main/java/org/apache/ignite/internal/catalog/descriptors/CatalogSchemaDescriptor.java: ## @@ -100,6 +100,11 @@ public CatalogSystemViewDescriptor[

Re: [PR] IGNITE-23748 Lock LWM when executing RO operation on data node [ignite-3]

2024-12-29 Thread via GitHub
rpuch merged PR #4974: URL: https://github.com/apache/ignite-3/pull/4974 -- 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...@ignit