Re: [PR] IGNITE-25609 WIP [ignite]

2025-06-11 Thread via GitHub
alex-plekhanov commented on code in PR #12130: URL: https://github.com/apache/ignite/pull/12130#discussion_r2139517119 ## modules/core/src/main/java/org/apache/ignite/client/ClientAuthorizationException.java: ## @@ -31,7 +31,12 @@ public class ClientAuthorizationException extend

Re: [PR] IGNITE-23495 .NET: Add cancellation to SQL and Compute APIs [ignite-3]

2025-06-11 Thread via GitHub
ptupitsyn merged PR #6018: URL: https://github.com/apache/ignite-3/pull/6018 -- 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] Bump org.bouncycastle:bcpkix-jdk18on from 1.80 to 1.81 [ignite-3]

2025-06-11 Thread via GitHub
ptupitsyn merged PR #5996: URL: https://github.com/apache/ignite-3/pull/5996 -- 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-25524 Fix GridLogCommandHandler path validation [ignite]

2025-06-11 Thread via GitHub
sonarqubecloud[bot] commented on PR #12136: URL: https://github.com/apache/ignite/pull/12136#issuecomment-2962207605 ## [![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-25524 Fix GridLogCommandHandler path validation [ignite]

2025-06-11 Thread via GitHub
sonarqubecloud[bot] commented on PR #12136: URL: https://github.com/apache/ignite/pull/12136#issuecomment-2962236552 ## [![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-24799 Eliminate ambiguity in '--snapshot create' output when w… [ignite]

2025-06-11 Thread via GitHub
sonarqubecloud[bot] commented on PR #11991: URL: https://github.com/apache/ignite/pull/11991#issuecomment-2961519160 ## [![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-25236 Calcite Engine. Reducing excessive log output on planning error [ignite]

2025-06-11 Thread via GitHub
alex-plekhanov merged PR #12047: URL: https://github.com/apache/ignite/pull/12047 -- 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-25642 Remove old node version from physical topology on readdition [ignite-3]

2025-06-11 Thread via GitHub
rpuch opened a new pull request, #6019: URL: https://github.com/apache/ignite-3/pull/6019 https://issues.apache.org/jira/browse/IGNITE-25642 -- 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 sp

Re: [PR] IGNITE-25609 WIP [ignite]

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

[PR] IGNITE-25650 Move WAL compacted file constants to NodeFileTree [ignite]

2025-06-11 Thread via GitHub
wernerdv opened a new pull request, #12135: URL: https://github.com/apache/ignite/pull/12135 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: ### Th

[PR] IGNITE-25524 Fix GridLogCommandHandler path validation [ignite]

2025-06-11 Thread via GitHub
ptupitsyn opened a new pull request, #12136: URL: https://github.com/apache/ignite/pull/12136 (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

[PR] IGNITE-25655 Fix TestJobExecutionStatusExecuting flakiness [ignite-3]

2025-06-11 Thread via GitHub
ptupitsyn opened a new pull request, #6020: URL: https://github.com/apache/ignite-3/pull/6020 (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-25593 Add Cluster API [ignite-3]

2025-06-11 Thread via GitHub
sk0x50 commented on code in PR #5986: URL: https://github.com/apache/ignite-3/pull/5986#discussion_r2139953644 ## modules/network-api/src/main/java/org/apache/ignite/internal/network/IgniteClusterImpl.java: ## @@ -0,0 +1,52 @@ +/* + * Licensed to the Apache Software Foundation (

[PR] IGNITE-25659 Remove VariableSource annotation [ignite-3]

2025-06-11 Thread via GitHub
sashapolo opened a new pull request, #6022: URL: https://github.com/apache/ignite-3/pull/6022 https://issues.apache.org/jira/browse/IGNITE-25659 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-25593 Add Cluster API [ignite-3]

2025-06-11 Thread via GitHub
sk0x50 commented on code in PR #5986: URL: https://github.com/apache/ignite-3/pull/5986#discussion_r2139942305 ## modules/client/src/main/java/org/apache/ignite/internal/client/network/ClientCluster.java: ## @@ -0,0 +1,71 @@ +/* + * Licensed to the Apache Software Foundation (AS

Re: [PR] IGNITE-25487 Schema aware JDBC metadata [ignite-3]

2025-06-11 Thread via GitHub
ygerzhedovich commented on code in PR #6011: URL: https://github.com/apache/ignite-3/pull/6011#discussion_r2139966831 ## modules/jdbc/src/integrationTest/java/org/apache/ignite/internal/jdbc/ItJdbcMetadataSelfTest.java: ## @@ -60,11 +57,17 @@ public class ItJdbcMetadataSelfTest

Re: [PR] IGNITE-25593 Add Cluster API [ignite-3]

2025-06-11 Thread via GitHub
ptupitsyn commented on code in PR #5986: URL: https://github.com/apache/ignite-3/pull/5986#discussion_r2139984709 ## modules/client/src/main/java/org/apache/ignite/internal/client/network/ClientCluster.java: ## @@ -0,0 +1,71 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] IGNITE-25593 Add Cluster API [ignite-3]

2025-06-11 Thread via GitHub
ptupitsyn commented on code in PR #5986: URL: https://github.com/apache/ignite-3/pull/5986#discussion_r2139979812 ## modules/network-api/src/main/java/org/apache/ignite/internal/network/IgniteClusterImpl.java: ## @@ -0,0 +1,52 @@ +/* + * Licensed to the Apache Software Foundatio

Re: [PR] IGNITE-25593 Add Cluster API [ignite-3]

2025-06-11 Thread via GitHub
sk0x50 commented on code in PR #5986: URL: https://github.com/apache/ignite-3/pull/5986#discussion_r2139990447 ## modules/api/src/main/java/org/apache/ignite/network/IgniteCluster.java: ## @@ -0,0 +1,48 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or mor

Re: [PR] IGNITE-25593 Add Cluster API [ignite-3]

2025-06-11 Thread via GitHub
sk0x50 commented on code in PR #5986: URL: https://github.com/apache/ignite-3/pull/5986#discussion_r2139995333 ## modules/network-api/src/main/java/org/apache/ignite/internal/network/IgniteClusterImpl.java: ## @@ -0,0 +1,52 @@ +/* + * Licensed to the Apache Software Foundation (

Re: [PR] IGNITE-25421 Add the requests throttling to raft client [ignite-3]

2025-06-11 Thread via GitHub
Cyrill commented on code in PR #5915: URL: https://github.com/apache/ignite-3/pull/5915#discussion_r2140007826 ## modules/transactions/src/main/java/org/apache/ignite/internal/tx/impl/WriteIntentSwitchProcessor.java: ## @@ -92,8 +92,9 @@ public CompletableFuture switchWriteInte

Re: [PR] IGNITE-25197 Move GridUnsafe to ignite-commons [ignite]

2025-06-11 Thread via GitHub
nizhikov commented on code in PR #12017: URL: https://github.com/apache/ignite/pull/12017#discussion_r2139992484 ## modules/commons/src/main/java/org/apache/ignite/IgniteCommonsSystemProperties.java: ## @@ -28,6 +28,9 @@ * These properties and variables can be used to affect t

Re: [PR] IGNITE-25632 Fix configuration compatibility [ignite-3]

2025-06-11 Thread via GitHub
Pochatkin merged PR #6014: URL: https://github.com/apache/ignite-3/pull/6014 -- 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-25421 Add the requests throttling to raft client [ignite-3]

2025-06-11 Thread via GitHub
Cyrill commented on code in PR #5915: URL: https://github.com/apache/ignite-3/pull/5915#discussion_r2140012722 ## modules/replicator/src/main/java/org/apache/ignite/internal/replicator/PlacementDriverMessageProcessor.java: ## @@ -151,7 +152,9 @@ CompletableFuture processPlaceme

Re: [PR] IGNITE-25593 Add Cluster API [ignite-3]

2025-06-11 Thread via GitHub
ptupitsyn commented on code in PR #5986: URL: https://github.com/apache/ignite-3/pull/5986#discussion_r2140013337 ## modules/api/src/main/java/org/apache/ignite/network/IgniteCluster.java: ## @@ -0,0 +1,48 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or

Re: [PR] IGNITE-25655 Fix TestJobExecutionStatusExecuting flakiness [ignite-3]

2025-06-11 Thread via GitHub
ptupitsyn merged PR #6020: URL: https://github.com/apache/ignite-3/pull/6020 -- 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-25661 Fix flaky test TcpDiscoverySpiMBeanTest#testNodeExclusion [ignite]

2025-06-11 Thread via GitHub
timoninmaxim opened a new pull request, #12137: URL: https://github.com/apache/ignite/pull/12137 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-25642 Remove old node version from physical topology on readdition [ignite-3]

2025-06-11 Thread via GitHub
rpuch merged PR #6019: URL: https://github.com/apache/ignite-3/pull/6019 -- 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-24818 Add tests to check lease time is present [ignite-3]

2025-06-11 Thread via GitHub
vldpyatkov commented on code in PR #6012: URL: https://github.com/apache/ignite-3/pull/6012#discussion_r2140899881 ## modules/transactions/src/integrationTest/java/org/apache/ignite/internal/disaster/ItDisasterRecoveryReconfigurationTest.java: ## @@ -1700,6 +1702,110 @@ void tes

[PR] IGNITE-25448 Reenable tests [ignite-3]

2025-06-11 Thread via GitHub
Cyrill opened a new pull request, #6027: URL: https://github.com/apache/ignite-3/pull/6027 https://issues.apache.org/jira/browse/IGNITE-25448 The timeouts that caused the tests to fail comes from the slow log4j loggers. If disabled, the tests start to pass -- This is an automated

Re: [PR] IGNITE-22662 Snapshot check refactoring [ignite]

2025-06-11 Thread via GitHub
sonarqubecloud[bot] commented on PR #11897: URL: https://github.com/apache/ignite/pull/11897#issuecomment-2962387844 ## [![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-25661 Fix flaky test TcpDiscoverySpiMBeanTest#testNodeExclusion [ignite]

2025-06-11 Thread via GitHub
sonarqubecloud[bot] commented on PR #12137: URL: https://github.com/apache/ignite/pull/12137#issuecomment-2962539068 ## [![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-25656 Improve node stop handling in SystemViewManagerImpl [ignite-3]

2025-06-11 Thread via GitHub
rpuch merged PR #6021: URL: https://github.com/apache/ignite-3/pull/6021 -- 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-25661 Fix flaky test TcpDiscoverySpiMBeanTest#testNodeExclusion [ignite]

2025-06-11 Thread via GitHub
sonarqubecloud[bot] commented on PR #12137: URL: https://github.com/apache/ignite/pull/12137#issuecomment-2962559417 ## [![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] WIP: un-mute test [ignite-3]

2025-06-11 Thread via GitHub
korlov42 closed pull request #5990: WIP: un-mute test URL: https://github.com/apache/ignite-3/pull/5990 -- 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:

Re: [PR] IGNITE-25659 Remove VariableSource annotation [ignite-3]

2025-06-11 Thread via GitHub
sashapolo merged PR #6022: URL: https://github.com/apache/ignite-3/pull/6022 -- 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-25487 Schema aware JDBC metadata [ignite-3]

2025-06-11 Thread via GitHub
xtern commented on code in PR #6011: URL: https://github.com/apache/ignite-3/pull/6011#discussion_r2140145745 ## modules/client-handler/src/main/java/org/apache/ignite/client/handler/requests/jdbc/JdbcMetadataCatalog.java: ## @@ -133,21 +147,22 @@ public CompletableFuture> getT

Re: [PR] IGNITE-25487 Schema aware JDBC metadata [ignite-3]

2025-06-11 Thread via GitHub
xtern commented on code in PR #6011: URL: https://github.com/apache/ignite-3/pull/6011#discussion_r2140146498 ## modules/jdbc/src/integrationTest/java/org/apache/ignite/internal/jdbc/ItJdbcMetadataSelfTest.java: ## @@ -60,11 +57,17 @@ public class ItJdbcMetadataSelfTest extends

Re: [PR] IGNITE-25487 Schema aware JDBC metadata [ignite-3]

2025-06-11 Thread via GitHub
xtern commented on code in PR #6011: URL: https://github.com/apache/ignite-3/pull/6011#discussion_r2140145745 ## modules/client-handler/src/main/java/org/apache/ignite/client/handler/requests/jdbc/JdbcMetadataCatalog.java: ## @@ -133,21 +147,22 @@ public CompletableFuture> getT

Re: [PR] IGNITE-25487 Schema aware JDBC metadata [ignite-3]

2025-06-11 Thread via GitHub
ygerzhedovich commented on code in PR #6011: URL: https://github.com/apache/ignite-3/pull/6011#discussion_r2139963945 ## modules/client-handler/src/main/java/org/apache/ignite/client/handler/requests/jdbc/JdbcMetadataCatalog.java: ## @@ -211,30 +230,31 @@ public CompletableFutur

Re: [PR] Bump org.junit.platform:junit-platform-testkit from 1.12.2 to 1.13.1 [ignite-3]

2025-06-11 Thread via GitHub
sashapolo commented on PR #6009: URL: https://github.com/apache/ignite-3/pull/6009#issuecomment-2962849130 @dependabot rebase -- 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] IGNITE-25369 Sql. Introduce EXPLAIN MAPPING FOR command [ignite-3]

2025-06-11 Thread via GitHub
zstan merged PR #6005: URL: https://github.com/apache/ignite-3/pull/6005 -- 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] Bump junit5 from 5.12.2 to 5.13.0 [ignite-3]

2025-06-11 Thread via GitHub
sashapolo commented on PR #5964: URL: https://github.com/apache/ignite-3/pull/5964#issuecomment-2962848532 @dependabot rebase -- 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.apache.commons:commons-lang3 from 3.14.0 to 3.17.0 [ignite-3]

2025-06-11 Thread via GitHub
sashapolo commented on PR #5965: URL: https://github.com/apache/ignite-3/pull/5965#issuecomment-2962289615 @dependabot rebase -- 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 junit5 from 5.12.2 to 5.13.0 [ignite-3]

2025-06-11 Thread via GitHub
sashapolo merged PR #5964: URL: https://github.com/apache/ignite-3/pull/5964 -- 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] Bump org.junit.platform:junit-platform-testkit from 1.12.2 to 1.13.1 [ignite-3]

2025-06-11 Thread via GitHub
sashapolo merged PR #6009: URL: https://github.com/apache/ignite-3/pull/6009 -- 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-25393 Sql. Unmute QueryTimeoutTest#testTimeoutDistributedRead [ignite-3]

2025-06-11 Thread via GitHub
korlov42 merged PR #6000: URL: https://github.com/apache/ignite-3/pull/6000 -- 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-25421 Add the requests throttling to raft client [ignite-3]

2025-06-11 Thread via GitHub
Cyrill commented on code in PR #5915: URL: https://github.com/apache/ignite-3/pull/5915#discussion_r2140012257 ## modules/replicator/src/main/java/org/apache/ignite/internal/replicator/ReplicaManager.java: ## @@ -446,7 +449,13 @@ private void handleReplicaRequest(ReplicaRequest

Re: [PR] IGNITE-25593 Add Cluster API [ignite-3]

2025-06-11 Thread via GitHub
sk0x50 commented on code in PR #5986: URL: https://github.com/apache/ignite-3/pull/5986#discussion_r2140061779 ## modules/api/src/main/java/org/apache/ignite/network/IgniteCluster.java: ## @@ -0,0 +1,48 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or mor

Re: [PR] IGNITE-25593 Add Cluster API [ignite-3]

2025-06-11 Thread via GitHub
ptupitsyn commented on code in PR #5986: URL: https://github.com/apache/ignite-3/pull/5986#discussion_r2140140973 ## modules/api/src/main/java/org/apache/ignite/network/IgniteCluster.java: ## @@ -0,0 +1,48 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or

Re: [PR] IGNITE-25317 Sql. Time, Timestamp, Timestamp LTZ. Cast Format. Sub-millsecond fractional format is broken [ignite-3]

2025-06-11 Thread via GitHub
lowka commented on code in PR #5977: URL: https://github.com/apache/ignite-3/pull/5977#discussion_r2140362912 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/exp/RexToLixTranslator.java: ## @@ -614,13 +614,7 @@ private Expression translateCastToDat

[PR] IGNITE-22177 add internal method for restarting partitions with cleanup [ignite-3]

2025-06-11 Thread via GitHub
alievmirza opened a new pull request, #6023: URL: https://github.com/apache/ignite-3/pull/6023 https://issues.apache.org/jira/browse/IGNITE-22177 Thank you for submitting the pull request. To streamline the review process of the patch and ensure better code quality we ask bot

[PR] Bump org.junit.platform:junit-platform-console from 1.13.0 to 1.13.1 [ignite-3]

2025-06-11 Thread via GitHub
dependabot[bot] opened a new pull request, #6025: URL: https://github.com/apache/ignite-3/pull/6025 Bumps [org.junit.platform:junit-platform-console](https://github.com/junit-team/junit5) from 1.13.0 to 1.13.1. Commits See full diff in https://github.com/junit-team/junit5/comm

[PR] Bump io.opentelemetry:opentelemetry-exporter-otlp from 1.50.0 to 1.51.0 [ignite-3]

2025-06-11 Thread via GitHub
dependabot[bot] opened a new pull request, #6024: URL: https://github.com/apache/ignite-3/pull/6024 Bumps [io.opentelemetry:opentelemetry-exporter-otlp](https://github.com/open-telemetry/opentelemetry-java) from 1.50.0 to 1.51.0. Release notes Sourced from https://github.com/open-

Re: [PR] IGNITE-25389 Explicit index cache path [ignite]

2025-06-11 Thread via GitHub
sonarqubecloud[bot] commented on PR #12133: URL: https://github.com/apache/ignite/pull/12133#issuecomment-2961959748 ## [![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-25389 Explicit index cache path [ignite]

2025-06-11 Thread via GitHub
sonarqubecloud[bot] commented on PR #12133: URL: https://github.com/apache/ignite/pull/12133#issuecomment-2962002022 ## [![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-25424 Fix flaky GridTcpCommunicationSpiConfigSelfTest [ignite]

2025-06-11 Thread via GitHub
timoninmaxim merged PR #12080: URL: https://github.com/apache/ignite/pull/12080 -- 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-23856 channel connection retry in ASYNC case [ignite]

2025-06-11 Thread via GitHub
timoninmaxim merged PR #11954: URL: https://github.com/apache/ignite/pull/11954 -- 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-23856 channel connection retry in ASYNC case [ignite]

2025-06-11 Thread via GitHub
sonarqubecloud[bot] commented on PR #11954: URL: https://github.com/apache/ignite/pull/11954#issuecomment-2962040768 ## [![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-25317 Sql. Time, Timestamp, Timestamp LTZ. Cast Format. Sub-millsecond fractional format is broken [ignite-3]

2025-06-11 Thread via GitHub
xtern commented on code in PR #5977: URL: https://github.com/apache/ignite-3/pull/5977#discussion_r2139740168 ## modules/sql-engine/src/integrationTest/sql/group1/types/date/test_incorrect_dates.test: ## @@ -101,7 +101,7 @@ statement error: Invalid DATE value, '1-00-01' INS

Re: [PR] IGNITE-24799 Eliminate ambiguity in '--snapshot create' output when w… [ignite]

2025-06-11 Thread via GitHub
alex-plekhanov merged PR #11991: URL: https://github.com/apache/ignite/pull/11991 -- 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-25604 Sql. Date/Time. Parser for time, date, timestamp types [ignite-3]

2025-06-11 Thread via GitHub
korlov42 merged PR #5998: URL: https://github.com/apache/ignite-3/pull/5998 -- 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-25236 Calcite Engine. Reducing excessive log output on planning error [ignite]

2025-06-11 Thread via GitHub
sonarqubecloud[bot] commented on PR #12047: URL: https://github.com/apache/ignite/pull/12047#issuecomment-2961587765 ## [![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-25634 Move WAL temp file constants to NodeFileTree [ignite]

2025-06-11 Thread via GitHub
nizhikov merged PR #12134: URL: https://github.com/apache/ignite/pull/12134 -- 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-25317 Sql. Time, Timestamp, Timestamp LTZ. Cast Format. Sub-millsecond fractional format is broken [ignite-3]

2025-06-11 Thread via GitHub
ygerzhedovich commented on code in PR #5977: URL: https://github.com/apache/ignite-3/pull/5977#discussion_r2139864923 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/exp/RexToLixTranslator.java: ## @@ -614,13 +614,7 @@ private Expression translateC

Re: [PR] IGNITE-25642 Remove old node version from physical topology on readdition [ignite-3]

2025-06-11 Thread via GitHub
tkalkirill commented on code in PR #6019: URL: https://github.com/apache/ignite-3/pull/6019#discussion_r2139855231 ## modules/network/src/main/java/org/apache/ignite/internal/network/scalecube/ScaleCubeTopologyService.java: ## @@ -88,64 +88,108 @@ void onMembershipEvent(Membersh

[PR] IGNITE-25656 Improve node stop handling in SystemViewManagerImpl [ignite-3]

2025-06-11 Thread via GitHub
rpuch opened a new pull request, #6021: URL: https://github.com/apache/ignite-3/pull/6021 https://issues.apache.org/jira/browse/IGNITE-25656 -- 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 sp

Re: [PR] IGNITE-25642 Remove old node version from physical topology on readdition [ignite-3]

2025-06-11 Thread via GitHub
rpuch commented on code in PR #6019: URL: https://github.com/apache/ignite-3/pull/6019#discussion_r2139871635 ## modules/network/src/main/java/org/apache/ignite/internal/network/scalecube/ScaleCubeTopologyService.java: ## @@ -88,64 +88,108 @@ void onMembershipEvent(MembershipEve

Re: [PR] IGNITE-25197 Move GridUnsafe to ignite-commons [ignite]

2025-06-11 Thread via GitHub
nizhikov commented on PR #12017: URL: https://github.com/apache/ignite/pull/12017#issuecomment-2962281215 After merging of PR we must do the following: 1. Remove `ReflectiveDirectBufferCleaner` - class not used. 2. Remove `DirectBufferCleaner` - interface has one implementation.

Re: [PR] IGNITE-25642 Remove old node version from physical topology on readdition [ignite-3]

2025-06-11 Thread via GitHub
rpuch commented on code in PR #6019: URL: https://github.com/apache/ignite-3/pull/6019#discussion_r2139872130 ## modules/network/src/main/java/org/apache/ignite/internal/network/scalecube/ScaleCubeTopologyService.java: ## @@ -88,64 +88,108 @@ void onMembershipEvent(MembershipEve

Re: [PR] IGNITE-25642 Remove old node version from physical topology on readdition [ignite-3]

2025-06-11 Thread via GitHub
rpuch commented on code in PR #6019: URL: https://github.com/apache/ignite-3/pull/6019#discussion_r2139873975 ## modules/network/src/main/java/org/apache/ignite/internal/network/scalecube/ScaleCubeTopologyService.java: ## @@ -88,64 +88,108 @@ void onMembershipEvent(MembershipEve

Re: [PR] IGNITE-25197 Move GridUnsafe to ignite-commons [ignite]

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

[PR] Bump com.github.spotbugs from 6.1.13 to 6.2.0 [ignite-3]

2025-06-11 Thread via GitHub
dependabot[bot] opened a new pull request, #6026: URL: https://github.com/apache/ignite-3/pull/6026 Bumps com.github.spotbugs from 6.1.13 to 6.2.0. [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=com.github.spot

Re: [PR] IGNITE-25567 Avoid starting with conflicting colocation status [ignite-3]

2025-06-11 Thread via GitHub
rpuch merged PR #6015: URL: https://github.com/apache/ignite-3/pull/6015 -- 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-24818 Add tests to check lease time is present [ignite-3]

2025-06-11 Thread via GitHub
Cyrill commented on code in PR #6012: URL: https://github.com/apache/ignite-3/pull/6012#discussion_r2141776302 ## modules/transactions/src/integrationTest/java/org/apache/ignite/internal/disaster/ItDisasterRecoveryReconfigurationTest.java: ## @@ -1700,6 +1702,110 @@ void testGra

Re: [PR] Bump org.junit.platform:junit-platform-console from 1.13.0 to 1.13.1 [ignite-3]

2025-06-11 Thread via GitHub
sashapolo merged PR #6025: URL: https://github.com/apache/ignite-3/pull/6025 -- 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] Bump com.github.spotbugs from 6.1.13 to 6.2.0 [ignite-3]

2025-06-11 Thread via GitHub
sashapolo merged PR #6026: URL: https://github.com/apache/ignite-3/pull/6026 -- 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] Bump io.opentelemetry:opentelemetry-exporter-otlp from 1.50.0 to 1.51.0 [ignite-3]

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