Re: [PR] IGNITE-25043 Support configuration rename [ignite-3]

2025-05-26 Thread via GitHub
ibessonov merged PR #5816: URL: https://github.com/apache/ignite-3/pull/5816 -- 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

[PR] IGNITE-25494 Improve FastTimestamps [ignite-3]

2025-05-26 Thread via GitHub
ibessonov opened a new pull request, #5900: URL: https://github.com/apache/ignite-3/pull/5900 https://issues.apache.org/jira/browse/IGNITE-25494 Thank you for submitting the pull request. To streamline the review process of the patch and ensure better code quality we ask both

Re: [PR] IGNITE-22090 Piggyback tx start in first tx request [ignite-3]

2025-05-26 Thread via GitHub
ascherbakoff closed pull request #5853: IGNITE-22090 Piggyback tx start in first tx request URL: https://github.com/apache/ignite-3/pull/5853 -- 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-25197 Move GridUnsafe to ignite-commons [ignite]

2025-05-26 Thread via GitHub
shishkovilja commented on code in PR #12017: URL: https://github.com/apache/ignite/pull/12017#discussion_r2108302012 ## modules/commons/src/main/java/org/apache/ignite/internal/util/CommonUtils.java: ## @@ -21,12 +21,13 @@ import java.lang.reflect.InvocationTargetException; im

Re: [PR] Bump jline from 3.29.0 to 3.30.3 [ignite-3]

2025-05-26 Thread via GitHub
ptupitsyn closed pull request #5884: Bump jline from 3.29.0 to 3.30.3 URL: https://github.com/apache/ignite-3/pull/5884 -- 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 unsub

Re: [PR] Bump org.openapi.generator from 7.12.0 to 7.13.0 [ignite-3]

2025-05-26 Thread via GitHub
ptupitsyn commented on PR #5731: URL: https://github.com/apache/ignite-3/pull/5731#issuecomment-2911196097 Compilation error. -- 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.

Re: [PR] Bump org.openapi.generator from 7.12.0 to 7.13.0 [ignite-3]

2025-05-26 Thread via GitHub
ptupitsyn closed pull request #5731: Bump org.openapi.generator from 7.12.0 to 7.13.0 URL: https://github.com/apache/ignite-3/pull/5731 -- 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 c

Re: [PR] Bump org.openapi.generator from 7.12.0 to 7.13.0 [ignite-3]

2025-05-26 Thread via GitHub
dependabot[bot] commented on PR #5731: URL: https://github.com/apache/ignite-3/pull/5731#issuecomment-2911196147 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, le

Re: [PR] Bump jline from 3.29.0 to 3.30.3 [ignite-3]

2025-05-26 Thread via GitHub
dependabot[bot] commented on PR #5884: URL: https://github.com/apache/ignite-3/pull/5884#issuecomment-2911194889 OK, I won't notify you again about this release, but will get in touch when a new version is available. You can also ignore all major, minor, or patch releases for a dependency b

[PR] IGNITE-25492 Fix NPE for DisasterRecoveryManager#localPartitionStates with collocation disabled [ignite-3]

2025-05-26 Thread via GitHub
tkalkirill opened a new pull request, #5899: URL: https://github.com/apache/ignite-3/pull/5899 https://issues.apache.org/jira/browse/IGNITE-25492 -- 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 t

Re: [PR] Bump io.swagger.core.v3:swagger-annotations from 2.2.30 to 2.2.32 [ignite-3]

2025-05-26 Thread via GitHub
ptupitsyn merged PR #5849: URL: https://github.com/apache/ignite-3/pull/5849 -- 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

[I] CREATE TABLE is synchronous but INSERT immediately after fails with "Table not found" [ignite]

2025-05-26 Thread via GitHub
hyemin-lee24 opened a new issue, #12098: URL: https://github.com/apache/ignite/issues/12098 ### Summary After executing a `CREATE TABLE` command via JDBC (through MyBatis in a Spring Boot application), an immediate `INSERT` operation fails with a "table not found" error. However, retryin

Re: [PR] IGNITE-25388 Several storage pathes per cache [ignite]

2025-05-26 Thread via GitHub
sonarqubecloud[bot] commented on PR #12081: URL: https://github.com/apache/ignite/pull/12081#issuecomment-2910550941 ## [![Quality Gate Failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-failed-20px.png 'Quality Gate Failed')](https://son

Re: [PR] IGNITE-25427 Move CdcConsumerState file logic to NodeFileTree [ignite]

2025-05-26 Thread via GitHub
nizhikov merged PR #12091: URL: https://github.com/apache/ignite/pull/12091 -- 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...@ig

Re: [PR] IGNITE-25427 Move CdcConsumerState file logic to NodeFileTree [ignite]

2025-05-26 Thread via GitHub
sonarqubecloud[bot] commented on PR #12091: URL: https://github.com/apache/ignite/pull/12091#issuecomment-2910431100 ## [![Quality Gate Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png 'Quality Gate Passed')](https://son

Re: [PR] IGNITE-25464 Fix double write intent switch application [ignite-3]

2025-05-26 Thread via GitHub
rpuch merged PR #5894: URL: https://github.com/apache/ignite-3/pull/5894 -- 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

Re: [PR] IGNITE-25491 Remove Message#fieldsCount [ignite]

2025-05-26 Thread via GitHub
sonarqubecloud[bot] commented on PR #12097: URL: https://github.com/apache/ignite/pull/12097#issuecomment-2910296049 ## [![Quality Gate Failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-failed-20px.png 'Quality Gate Failed')](https://son

[PR] IGNITE-25491 Remove Message#fieldsCount [ignite]

2025-05-26 Thread via GitHub
timoninmaxim opened a new pull request, #12097: URL: https://github.com/apache/ignite/pull/12097 Thank you for submitting the pull request to the Apache Ignite. In order to streamline the review of the contribution we ask you to ensure the following steps have been taken: ##

Re: [PR] IGNITE-25473 .NET: Fix compute executor type resolver compatibility [ignite-3]

2025-05-26 Thread via GitHub
ptupitsyn merged PR #5898: URL: https://github.com/apache/ignite-3/pull/5898 -- 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

Re: [I] Apache Ignite 3.0 - Reading Data from ignite via C++ Client [ignite]

2025-05-26 Thread via GitHub
isapego commented on issue #12070: URL: https://github.com/apache/ignite/issues/12070#issuecomment-2910030982 I've filed a ticket to implement transaction timeouts for C++ Client: https://issues.apache.org/jira/browse/IGNITE-25488 -- This is an automated message from the Apache Git Servic

[PR] IGNITE-25473 .NET: Fix compute executor type resolver [ignite-3]

2025-05-26 Thread via GitHub
ptupitsyn opened a new pull request, #5898: URL: https://github.com/apache/ignite-3/pull/5898 (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: [I] Apache Ignite 3.0 - Reading Data from ignite via C++ Client [ignite]

2025-05-26 Thread via GitHub
isapego commented on issue #12070: URL: https://github.com/apache/ignite/issues/12070#issuecomment-2909966949 @hsadia538 The exception is still the same? P.S. The `result.current_page()` method is just a getter and can not fail, you don't have to surround it with try-catch. -- This is

Re: [PR] IGNITE-22090 Piggyback tx start in first tx request [ignite-3]

2025-05-26 Thread via GitHub
ascherbakoff commented on code in PR #5853: URL: https://github.com/apache/ignite-3/pull/5853#discussion_r2107476136 ## modules/client/src/main/java/org/apache/ignite/internal/client/compute/ClientJobExecution.java: ## @@ -131,11 +131,10 @@ public ClusterNode node() { /

Re: [PR] IGNITE-22090 Piggyback tx start in first tx request [ignite-3]

2025-05-26 Thread via GitHub
ascherbakoff commented on code in PR #5853: URL: https://github.com/apache/ignite-3/pull/5853#discussion_r2107475776 ## modules/client/src/main/java/org/apache/ignite/internal/client/tx/ClientLazyTransaction.java: ## @@ -135,41 +140,52 @@ public String nodeName() { *

Re: [PR] IGNITE-20347 Add transaction id to commitWrite and abortWrite methods of Storage [ignite-3]

2025-05-26 Thread via GitHub
tkalkirill merged PR #5875: URL: https://github.com/apache/ignite-3/pull/5875 -- 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...@

[PR] IGNITE-14777: row_number() with window functions unsupported [ignite]

2025-05-26 Thread via GitHub
oleg-zinovev opened a new pull request, #12096: URL: https://github.com/apache/ignite/pull/12096 Thank you for submitting the pull request to the Apache Ignite. In order to streamline the review of the contribution we ask you to ensure the following steps have been taken: ##

Re: [PR] IGNITE-22090 Piggyback tx start in first tx request [ignite-3]

2025-05-26 Thread via GitHub
denis-chudov commented on code in PR #5853: URL: https://github.com/apache/ignite-3/pull/5853#discussion_r2107203195 ## modules/client/src/main/java/org/apache/ignite/internal/client/compute/ClientJobExecution.java: ## @@ -117,11 +118,10 @@ public ClusterNode node() { /

Re: [PR] IGNITE-24535 Reduce the number of JUnit parameters in SnapshotCompatibilityTest, refactor corresponding code [ignite]

2025-05-26 Thread via GitHub
chesnokoff commented on code in PR #11969: URL: https://github.com/apache/ignite/pull/11969#discussion_r2106808218 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -80,201 +80,161 @@ public class SnapshotCom

[PR] IGNITE-24535 Introduce NodeFileTreeCompatibilityTest add extracted common code to SnapshotCompatibilityAbstractTest [ignite]

2025-05-26 Thread via GitHub
vladnovoren opened a new pull request, #12095: URL: https://github.com/apache/ignite/pull/12095 Thank you for submitting the pull request to the Apache Ignite. In order to streamline the review of the contribution we ask you to ensure the following steps have been taken: ###

Re: [PR] IGNITE-20347 Add transaction id to commitWrite and abortWrite methods of Storage [ignite-3]

2025-05-26 Thread via GitHub
tkalkirill commented on code in PR #5875: URL: https://github.com/apache/ignite-3/pull/5875#discussion_r2107195663 ## modules/storage-api/src/main/java/org/apache/ignite/internal/storage/AbortResult.java: ## @@ -0,0 +1,86 @@ +/* + * Licensed to the Apache Software Foundation (AS

Re: [PR] IGNITE-20347 Add transaction id to commitWrite and abortWrite methods of Storage [ignite-3]

2025-05-26 Thread via GitHub
tkalkirill commented on code in PR #5875: URL: https://github.com/apache/ignite-3/pull/5875#discussion_r2107195663 ## modules/storage-api/src/main/java/org/apache/ignite/internal/storage/AbortResult.java: ## @@ -0,0 +1,86 @@ +/* + * Licensed to the Apache Software Foundation (AS

Re: [PR] IGNITE-25447 Use annotation to conditionally disable mismatching colocation tests [ignite-3]

2025-05-26 Thread via GitHub
sanpwc merged PR #5873: URL: https://github.com/apache/ignite-3/pull/5873 -- 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...@igni

Re: [PR] IGNITE-25481 Fix NPE in ItDisasterRecoveryReconfigurationTest#testIncompleteRebalanceBeforeAutomaticResetPartitions [ignite-3]

2025-05-26 Thread via GitHub
sanpwc merged PR #5897: URL: https://github.com/apache/ignite-3/pull/5897 -- 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...@igni

Re: [I] Apache Ignite 3.0 - Reading Data from ignite via C++ Client [ignite]

2025-05-26 Thread via GitHub
hsadia538 commented on issue #12070: URL: https://github.com/apache/ignite/issues/12070#issuecomment-2909462244 I still cant seem to read data with the cpp app. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

Re: [PR] IGNITE-20347 Add transaction id to commitWrite and abortWrite methods of Storage [ignite-3]

2025-05-26 Thread via GitHub
rpuch commented on code in PR #5875: URL: https://github.com/apache/ignite-3/pull/5875#discussion_r2107007110 ## modules/storage-api/src/main/java/org/apache/ignite/internal/storage/CommitResultStatus.java: ## @@ -0,0 +1,30 @@ +/* + * Licensed to the Apache Software Foundation (

Re: [PR] IGNITE-23856 channel connection retry in ASYNC case [ignite]

2025-05-26 Thread via GitHub
timoninmaxim commented on code in PR #11954: URL: https://github.com/apache/ignite/pull/11954#discussion_r2106858938 ## modules/core/src/main/java/org/apache/ignite/internal/client/thin/ReliableChannel.java: ## @@ -243,65 +243,105 @@ private void handleServiceAsync( Li

Re: [PR] IGNITE-25425 Dynamically compute data region sizes [ignite-3]

2025-05-26 Thread via GitHub
sashapolo merged PR #5886: URL: https://github.com/apache/ignite-3/pull/5886 -- 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

Re: [PR] IGNITE-25427 Move CdcConsumerState file logic to NodeFileTree [ignite]

2025-05-26 Thread via GitHub
sonarqubecloud[bot] commented on PR #12091: URL: https://github.com/apache/ignite/pull/12091#issuecomment-2909336371 ## [![Quality Gate Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png 'Quality Gate Passed')](https://son

Re: [PR] IGNITE-24535 Reduce the number of JUnit parameters in SnapshotCompatibilityTest, refactor corresponding code [ignite]

2025-05-26 Thread via GitHub
chesnokoff commented on code in PR #11969: URL: https://github.com/apache/ignite/pull/11969#discussion_r2106813522 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -80,201 +80,161 @@ public class SnapshotCom

Re: [PR] IGNITE-25425 Dynamically compute data region sizes [ignite-3]

2025-05-26 Thread via GitHub
ibessonov commented on code in PR #5886: URL: https://github.com/apache/ignite-3/pull/5886#discussion_r2106734796 ## modules/configuration/src/main/java/org/apache/ignite/internal/configuration/ConfigurationRegistry.java: ## @@ -118,6 +124,12 @@ public InnerNode createRootNode(R

Re: [PR] IGNITE-25427 Move CdcConsumerState file logic to NodeFileTree [ignite]

2025-05-26 Thread via GitHub
nizhikov commented on code in PR #12091: URL: https://github.com/apache/ignite/pull/12091#discussion_r2107041236 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/filename/NodeFileTree.java: ## @@ -246,6 +244,9 @@ public class NodeFileTree ext

Re: [PR] IGNITE-25427 Move CdcConsumerState file logic to NodeFileTree [ignite]

2025-05-26 Thread via GitHub
nizhikov commented on code in PR #12091: URL: https://github.com/apache/ignite/pull/12091#discussion_r2107039182 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/filename/NodeFileTree.java: ## @@ -49,6 +49,7 @@ import static org.apache.igni

Re: [PR] IGNITE-25427 Move CdcConsumerState file logic to NodeFileTree [ignite]

2025-05-26 Thread via GitHub
nizhikov commented on code in PR #12091: URL: https://github.com/apache/ignite/pull/12091#discussion_r2107027984 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/filename/NodeFileTree.java: ## @@ -246,6 +244,9 @@ public class NodeFileTree ext

Re: [PR] IGNITE-25428 Move defragmentation logic to CacheFileTree [ignite]

2025-05-26 Thread via GitHub
nizhikov commented on code in PR #12094: URL: https://github.com/apache/ignite/pull/12094#discussion_r2106729929 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/file/FilePageStoreManager.java: ## @@ -640,9 +624,15 @@ private CacheStoreHolder

Re: [PR] IGNITE-23856 channel connection retry in ASYNC case [ignite]

2025-05-26 Thread via GitHub
FewJuho commented on code in PR #11954: URL: https://github.com/apache/ignite/pull/11954#discussion_r2106948830 ## modules/core/src/main/java/org/apache/ignite/internal/client/thin/ReliableChannel.java: ## @@ -243,65 +243,105 @@ private void handleServiceAsync( List fa

Re: [PR] IGNITE-24535 Reduce the number of JUnit parameters in SnapshotCompatibilityTest, refactor corresponding code [ignite]

2025-05-26 Thread via GitHub
chesnokoff commented on code in PR #11969: URL: https://github.com/apache/ignite/pull/11969#discussion_r2106818506 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -80,201 +80,161 @@ public class SnapshotCom

Re: [PR] IGNITE-25428 Move defragmentation logic to CacheFileTree [ignite]

2025-05-26 Thread via GitHub
nizhikov merged PR #12094: URL: https://github.com/apache/ignite/pull/12094 -- 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...@ig

Re: [PR] IGNITE-24535 Reduce the number of JUnit parameters in SnapshotCompatibilityTest, refactor corresponding code [ignite]

2025-05-26 Thread via GitHub
chesnokoff commented on code in PR #11969: URL: https://github.com/apache/ignite/pull/11969#discussion_r2106789939 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -450,12 +469,36 @@ public void checkCaches(

Re: [PR] IGNITE-25425 Dynamically compute data region sizes [ignite-3]

2025-05-26 Thread via GitHub
ibessonov commented on code in PR #5886: URL: https://github.com/apache/ignite-3/pull/5886#discussion_r2106923881 ## modules/configuration/src/main/java/org/apache/ignite/internal/configuration/ConfigurationRegistry.java: ## @@ -118,6 +124,12 @@ public InnerNode createRootNode(R

Re: [PR] IGNITE-25427 Move CdcConsumerState file logic to NodeFileTree [ignite]

2025-05-26 Thread via GitHub
nizhikovprivate commented on code in PR #12091: URL: https://github.com/apache/ignite/pull/12091#discussion_r2106911983 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/filename/NodeFileTree.java: ## @@ -879,6 +903,51 @@ public static int typ

Re: [PR] IGNITE-25424 Fix flaky GridTcpCommunicationSpiConfigSelfTest [ignite]

2025-05-26 Thread via GitHub
sonarqubecloud[bot] commented on PR #12080: URL: https://github.com/apache/ignite/pull/12080#issuecomment-2909035969 ## [![Quality Gate Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png 'Quality Gate Passed')](https://son

Re: [PR] IGNITE-25466 .NET: Update NuGet README with platform compute info [ignite-3]

2025-05-26 Thread via GitHub
ptupitsyn merged PR #5889: URL: https://github.com/apache/ignite-3/pull/5889 -- 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

Re: [PR] IGNITE-24535 Reduce the number of JUnit parameters in SnapshotCompatibilityTest, refactor corresponding code [ignite]

2025-05-26 Thread via GitHub
chesnokoff commented on code in PR #11969: URL: https://github.com/apache/ignite/pull/11969#discussion_r2106806485 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -450,12 +469,36 @@ public void checkCaches(

Re: [PR] IGNITE-25469 Perform cleanup in a separate thread pool [ignite-3]

2025-05-26 Thread via GitHub
sanpwc merged PR #5891: URL: https://github.com/apache/ignite-3/pull/5891 -- 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...@igni

Re: [PR] IGNITE-23856 channel connection retry in ASYNC case [ignite]

2025-05-26 Thread via GitHub
timoninmaxim commented on code in PR #11954: URL: https://github.com/apache/ignite/pull/11954#discussion_r2106862471 ## modules/core/src/main/java/org/apache/ignite/internal/client/thin/ReliableChannel.java: ## @@ -243,63 +243,120 @@ private void handleServiceAsync( Li

[PR] IGNITE-25481 Fix NPE in ItDisasterRecoveryReconfigurationTest#testIncompleteRebalanceBeforeAutomaticResetPartitions [ignite-3]

2025-05-26 Thread via GitHub
sanpwc opened a new pull request, #5897: URL: https://github.com/apache/ignite-3/pull/5897 https://issues.apache.org/jira/browse/IGNITE-25481 -- 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 s

Re: [PR] IGNITE-24535 Reduce the number of JUnit parameters in SnapshotCompatibilityTest, refactor corresponding code [ignite]

2025-05-26 Thread via GitHub
chesnokoff commented on code in PR #11969: URL: https://github.com/apache/ignite/pull/11969#discussion_r2106827269 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -80,201 +80,161 @@ public class SnapshotCom

Re: [PR] IGNITE-25200 Fix incorrect char[] to String conversion in GridCommandHandlerFactoryAbstractTest$JmxCommandHandler#toString (password handling) [ignite]

2025-05-26 Thread via GitHub
sonarqubecloud[bot] commented on PR #12018: URL: https://github.com/apache/ignite/pull/12018#issuecomment-290176 ## [![Quality Gate Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png 'Quality Gate Passed')](https://son

Re: [PR] IGNITE-23856 channel connection retry in ASYNC case [ignite]

2025-05-26 Thread via GitHub
timoninmaxim commented on code in PR #11954: URL: https://github.com/apache/ignite/pull/11954#discussion_r2106854127 ## modules/core/src/main/java/org/apache/ignite/internal/client/thin/ReliableChannel.java: ## @@ -243,65 +243,105 @@ private void handleServiceAsync( Li

Re: [PR] IGNITE-25436 Sql. Explain. Improve test coverage of modify operators [ignite-3]

2025-05-26 Thread via GitHub
korlov42 merged PR #5868: URL: https://github.com/apache/ignite-3/pull/5868 -- 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...@ig

Re: [PR] IGNITE-25425 Dynamically compute data region sizes [ignite-3]

2025-05-26 Thread via GitHub
sashapolo commented on code in PR #5886: URL: https://github.com/apache/ignite-3/pull/5886#discussion_r2106779391 ## modules/configuration/src/main/java/org/apache/ignite/internal/configuration/ConfigurationRegistry.java: ## @@ -118,6 +124,12 @@ public InnerNode createRootNode(R

Re: [PR] IGNITE-25200 Fix incorrect char[] to String conversion in GridCommandHandlerFactoryAbstractTest$JmxCommandHandler#toString (password handling) [ignite]

2025-05-26 Thread via GitHub
petrov-mg merged PR #12018: URL: https://github.com/apache/ignite/pull/12018 -- 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

Re: [PR] IGNITE-25428 Move defragmentation logic to CacheFileTree [ignite]

2025-05-26 Thread via GitHub
sonarqubecloud[bot] commented on PR #12094: URL: https://github.com/apache/ignite/pull/12094#issuecomment-2908876243 ## [![Quality Gate Failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-failed-20px.png 'Quality Gate Failed')](https://son

Re: [PR] IGNITE-25428 Move defragmentation logic to CacheFileTree [ignite]

2025-05-26 Thread via GitHub
nizhikov commented on code in PR #12094: URL: https://github.com/apache/ignite/pull/12094#discussion_r2106734174 ## modules/core/src/test/java/org/apache/ignite/internal/processors/cache/persistence/filename/FileTreeTestUtils.java: ## @@ -0,0 +1,32 @@ +/* + * Licensed to the Apa