[PR] IGNITE-23403 Remove Unused error codes [ignite-3]

2024-11-18 Thread via GitHub
tkalkirill opened a new pull request, #4742: URL: https://github.com/apache/ignite-3/pull/4742 https://issues.apache.org/jira/browse/IGNITE-23403 -- 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] IGNITE-22940 Expose partition size to public API [ignite-3]

2024-11-18 Thread via GitHub
tkalkirill merged PR #4733: URL: https://github.com/apache/ignite-3/pull/4733 -- 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-23700 Fix NPE in AbstractTopologyAwareGroupServiceTest [ignite-3]

2024-11-18 Thread via GitHub
sanpwc opened a new pull request, #4743: URL: https://github.com/apache/ignite-3/pull/4743 https://issues.apache.org/jira/browse/IGNITE-23700 -- 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-23623 Fix GridCacheAbstractMetricsSelfTest [ignite]

2024-11-18 Thread via GitHub
timoninmaxim commented on code in PR #11658: URL: https://github.com/apache/ignite/pull/11658#discussion_r1846210211 ## modules/core/src/test/java/org/apache/ignite/internal/processors/cache/GridCacheAbstractSelfTest.java: ## @@ -150,8 +150,8 @@ else if (isMultiJvm() && !(storeS

Re: [PR] Bump docfx from 2.77.0 to 2.78.0 in /modules/platforms/dotnet [ignite-3]

2024-11-18 Thread via GitHub
ptupitsyn merged PR #4739: URL: https://github.com/apache/ignite-3/pull/4739 -- 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 Microsoft.CodeAnalysis.NetAnalyzers from 8.0.0 to 9.0.0 in /modules/platforms/dotnet [ignite-3]

2024-11-18 Thread via GitHub
dependabot[bot] closed pull request #4726: Bump Microsoft.CodeAnalysis.NetAnalyzers from 8.0.0 to 9.0.0 in /modules/platforms/dotnet URL: https://github.com/apache/ignite-3/pull/4726 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

Re: [PR] Bump Microsoft.CodeAnalysis.NetAnalyzers from 8.0.0 to 9.0.0 in /modules/platforms/dotnet [ignite-3]

2024-11-18 Thread via GitHub
dependabot[bot] commented on PR #4726: URL: https://github.com/apache/ignite-3/pull/4726#issuecomment-2483061952 OK, I won't notify you about version 9.x.x again, unless you re-open this PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] IGNITE-22940 Expose partition size to public API [ignite-3]

2024-11-18 Thread via GitHub
rpuch commented on code in PR #4733: URL: https://github.com/apache/ignite-3/pull/4733#discussion_r1846373016 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/disaster/DisasterRecoverySystemViews.java: ## @@ -65,6 +66,7 @@ static SystemView createLoca

Re: [PR] IGNITE-23587 Make PlacementDriver able to choose lease candidate from pendings [ignite-3]

2024-11-18 Thread via GitHub
denis-chudov commented on code in PR #4707: URL: https://github.com/apache/ignite-3/pull/4707#discussion_r1846159797 ## modules/core/src/main/java/org/apache/ignite/internal/util/CollectionUtils.java: ## @@ -171,6 +171,33 @@ public static Set union(@Nullable Set set, @Nullable

Re: [PR] IGNITE-21810 Migrate cpp client/odbc build to Conan 2 [ignite-3]

2024-11-18 Thread via GitHub
dant3 closed pull request #3465: IGNITE-21810 Migrate cpp client/odbc build to Conan 2 URL: https://github.com/apache/ignite-3/pull/3465 -- 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

Re: [PR] IGNITE-23587 Make PlacementDriver able to choose lease candidate from pendings [ignite-3]

2024-11-18 Thread via GitHub
JAkutenshi commented on code in PR #4707: URL: https://github.com/apache/ignite-3/pull/4707#discussion_r1846397897 ## modules/core/src/main/java/org/apache/ignite/internal/util/CollectionUtils.java: ## @@ -171,6 +171,33 @@ public static Set union(@Nullable Set set, @Nullable T

Re: [PR] IGNITE-23076 Add shared timeout worker for all client channels [ignite-3]

2024-11-18 Thread via GitHub
ptupitsyn merged PR #4730: URL: https://github.com/apache/ignite-3/pull/4730 -- 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-22940 Expose partition size to public API [ignite-3]

2024-11-18 Thread via GitHub
rpuch commented on code in PR #4733: URL: https://github.com/apache/ignite-3/pull/4733#discussion_r1846589086 ## modules/table/src/integrationTest/java/org/apache/ignite/internal/disaster/ItDisasterRecoverySystemViewTest.java: ## @@ -178,4 +186,21 @@ private static int getTableI

Re: [PR] Bump Microsoft.Extensions.Logging.Console from 8.0.1 to 9.0.0 in /modules/platforms/dotnet [ignite-3]

2024-11-18 Thread via GitHub
ptupitsyn merged PR #4718: URL: https://github.com/apache/ignite-3/pull/4718 -- 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-23441 Sql. Cancellation of script execution [ignite-3]

2024-11-18 Thread via GitHub
xtern commented on code in PR #4706: URL: https://github.com/apache/ignite-3/pull/4706#discussion_r1846505037 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/SqlQueryProcessor.java: ## @@ -543,7 +543,7 @@ public int openedCursors() { }

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

2024-11-18 Thread via GitHub
chesnokoff commented on code in PR #11658: URL: https://github.com/apache/ignite/pull/11658#discussion_r1846645851 ## modules/core/src/test/java/org/apache/ignite/internal/processors/cache/GridCacheAbstractSelfTest.java: ## @@ -150,8 +150,8 @@ else if (isMultiJvm() && !(storeStg

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

2024-11-18 Thread via GitHub
chesnokoff commented on code in PR #11658: URL: https://github.com/apache/ignite/pull/11658#discussion_r1846645851 ## modules/core/src/test/java/org/apache/ignite/internal/processors/cache/GridCacheAbstractSelfTest.java: ## @@ -150,8 +150,8 @@ else if (isMultiJvm() && !(storeStg

[PR] IGNITE-23657 Extend logging of IdleVerify [ignite]

2024-11-18 Thread via GitHub
J-Bakuli opened a new pull request, #11665: URL: https://github.com/apache/ignite/pull/11665 To extend the list of idle verify process events. At the moment there is only an indirect mentioning of 'VerifyBackupPartitions' in Checkpoint event. It includes info regarding the start of i

Re: [PR] Bump org.openapi.generator from 7.9.0 to 7.10.0 [ignite-3]

2024-11-18 Thread via GitHub
ptupitsyn commented on PR #4741: URL: https://github.com/apache/ignite-3/pull/4741#issuecomment-2483815649 @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.openapi.generator from 7.9.0 to 7.10.0 [ignite-3]

2024-11-18 Thread via GitHub
ptupitsyn merged PR #4741: URL: https://github.com/apache/ignite-3/pull/4741 -- 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-23691 Disable ItTableRaftSnapshotsTest.txSemanticsIsMaintained [ignite-3]

2024-11-18 Thread via GitHub
rpuch opened a new pull request, #4734: URL: https://github.com/apache/ignite-3/pull/4734 https://issues.apache.org/jira/browse/IGNITE-23691 -- 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

[PR] Bump org.openapi.generator from 7.9.0 to 7.10.0 [ignite-3]

2024-11-18 Thread via GitHub
dependabot[bot] opened a new pull request, #4741: URL: https://github.com/apache/ignite-3/pull/4741 Bumps org.openapi.generator from 7.9.0 to 7.10.0. [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=org.openapi.g

[PR] Bump docfx from 2.77.0 to 2.78.0 in /modules/platforms/dotnet [ignite-3]

2024-11-18 Thread via GitHub
dependabot[bot] opened a new pull request, #4739: URL: https://github.com/apache/ignite-3/pull/4739 Bumps [docfx](https://github.com/dotnet/docfx) from 2.77.0 to 2.78.0. Release notes Sourced from https://github.com/dotnet/docfx/releases";>docfx's releases. v2.78.0 Thi

Re: [PR] IGNITE-22940 Expose partition size to public API [ignite-3]

2024-11-18 Thread via GitHub
tkalkirill commented on code in PR #4733: URL: https://github.com/apache/ignite-3/pull/4733#discussion_r1846688157 ## modules/table/src/integrationTest/java/org/apache/ignite/internal/disaster/ItDisasterRecoverySystemViewTest.java: ## @@ -131,7 +135,11 @@ void testLocalPartitio

[PR] IGNITE-23429 Support cancellation tokens in IgniteCompute [ignite-3]

2024-11-18 Thread via GitHub
zstan opened a new pull request, #4738: URL: https://github.com/apache/ignite-3/pull/4738 Thank you for submitting the pull request. To streamline the review process of the patch and ensure better code quality we ask both an author and a reviewer to verify the following: ###

[PR] Bump io.swagger.core.v3:swagger-annotations from 2.2.25 to 2.2.26 [ignite-3]

2024-11-18 Thread via GitHub
dependabot[bot] opened a new pull request, #4740: URL: https://github.com/apache/ignite-3/pull/4740 Bumps io.swagger.core.v3:swagger-annotations from 2.2.25 to 2.2.26. [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-

Re: [PR] IGNITE-23429 Support cancellation tokens in IgniteCompute [ignite-3]

2024-11-18 Thread via GitHub
valepakh commented on code in PR #4738: URL: https://github.com/apache/ignite-3/pull/4738#discussion_r1846923992 ## modules/compute/src/integrationTest/java/org/apache/ignite/internal/compute/ItComputeBaseTest.java: ## @@ -379,6 +389,168 @@ void executeColocatedThrowsTableNotFo

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

2024-11-18 Thread via GitHub
ptupitsyn commented on PR #4740: URL: https://github.com/apache/ignite-3/pull/4740#issuecomment-2483815792 @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-23569 Fix TcpClientChannel.close reliability [ignite-3]

2024-11-18 Thread via GitHub
ptupitsyn merged PR #4737: URL: https://github.com/apache/ignite-3/pull/4737 -- 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 docfx from 2.77.0 to 2.78.0 in /modules/platforms/dotnet [ignite-3]

2024-11-18 Thread via GitHub
ptupitsyn commented on PR #4739: URL: https://github.com/apache/ignite-3/pull/4739#issuecomment-2483815872 @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-23138 Fix TxWithKeyContentionSelfTest (Cache 12 group) [ignite]

2024-11-18 Thread via GitHub
ivandasch merged PR #11512: URL: https://github.com/apache/ignite/pull/11512 -- 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-23679 [ducktests] Use dependencies from the ignite instance under test in test runtime [ignite]

2024-11-18 Thread via GitHub
skorotkov opened a new pull request, #11664: URL: https://github.com/apache/ignite/pull/11664 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: ### T

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

2024-11-18 Thread via GitHub
ptupitsyn merged PR #4740: URL: https://github.com/apache/ignite-3/pull/4740 -- 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-23076 Add shared timeout worker for all client channels [ignite-3]

2024-11-18 Thread via GitHub
vldpyatkov commented on code in PR #4730: URL: https://github.com/apache/ignite-3/pull/4730#discussion_r1846078959 ## modules/client/src/main/java/org/apache/ignite/internal/client/ClientTimeoutWorker.java: ## @@ -0,0 +1,90 @@ +/* + * Licensed to the Apache Software Foundation (

Re: [PR] IGNITE-23076 Add shared timeout worker for all client channels [ignite-3]

2024-11-18 Thread via GitHub
ptupitsyn commented on code in PR #4730: URL: https://github.com/apache/ignite-3/pull/4730#discussion_r1846420537 ## modules/client/src/main/java/org/apache/ignite/internal/client/ClientTimeoutWorker.java: ## @@ -0,0 +1,90 @@ +/* + * Licensed to the Apache Software Foundation (A

Re: [PR] IGNITE-23683 Support format versioning when storing lease data in RocksDB partitions [ignite-3]

2024-11-18 Thread via GitHub
rpuch merged PR #4732: URL: https://github.com/apache/ignite-3/pull/4732 -- 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-23429 Support cancellation tokens in IgniteCompute [ignite-3]

2024-11-18 Thread via GitHub
valepakh commented on code in PR #4738: URL: https://github.com/apache/ignite-3/pull/4738#discussion_r1846921084 ## modules/compute/src/integrationTest/java/org/apache/ignite/internal/compute/ItComputeBaseTest.java: ## @@ -379,6 +389,168 @@ void executeColocatedThrowsTableNotFo

[PR] IGNITE-23569 Fix TcpClientChannel.close reliability [ignite-3]

2024-11-18 Thread via GitHub
ptupitsyn opened a new pull request, #4737: URL: https://github.com/apache/ignite-3/pull/4737 (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] Bump Microsoft.CodeAnalysis.NetAnalyzers from 8.0.0 to 9.0.0 in /modules/platforms/dotnet [ignite-3]

2024-11-18 Thread via GitHub
ptupitsyn commented on PR #4726: URL: https://github.com/apache/ignite-3/pull/4726#issuecomment-2483061755 @dependabot ignore this major version -- 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 th

Re: [PR] IGNITE-23587 Make PlacementDriver able to choose lease candidate from pendings [ignite-3]

2024-11-18 Thread via GitHub
JAkutenshi commented on code in PR #4707: URL: https://github.com/apache/ignite-3/pull/4707#discussion_r1846380551 ## modules/core/src/main/java/org/apache/ignite/internal/util/CollectionUtils.java: ## @@ -171,6 +171,33 @@ public static Set union(@Nullable Set set, @Nullable T

Re: [PR] Run Sonar with JDK17 [ignite]

2024-11-18 Thread via GitHub
timoninmaxim closed pull request #10899: Run Sonar with JDK17 URL: https://github.com/apache/ignite/pull/10899 -- 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-21285 Fix conflict version for cdc DataEntries [ignite]

2024-11-18 Thread via GitHub
timoninmaxim closed pull request #11184: IGNITE-21285 Fix conflict version for cdc DataEntries URL: https://github.com/apache/ignite/pull/11184 -- 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-23697 Fix negative pin avgWriteSpeed in message Checkpoint finished [ignite-3]

2024-11-18 Thread via GitHub
tkalkirill merged PR #4735: URL: https://github.com/apache/ignite-3/pull/4735 -- 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: [I] Ignite as a cache for SQL Server database [ignite]

2024-11-18 Thread via GitHub
sk0x50 closed issue #11622: Ignite as a cache for SQL Server database URL: https://github.com/apache/ignite/issues/11622 -- 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 unsu

[PR] IGNITE-23697 Fix negative pin avgWriteSpeed in message Checkpoint finished [ignite-3]

2024-11-18 Thread via GitHub
tkalkirill opened a new pull request, #4735: URL: https://github.com/apache/ignite-3/pull/4735 https://issues.apache.org/jira/browse/IGNITE-23697 -- 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] IGNITE-23689 Add tests for standalone compute [ignite-3]

2024-11-18 Thread via GitHub
Pochatkin merged PR #4731: URL: https://github.com/apache/ignite-3/pull/4731 -- 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-23687 Add missing tests [ignite-3]

2024-11-18 Thread via GitHub
Pochatkin merged PR #4729: URL: https://github.com/apache/ignite-3/pull/4729 -- 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-23696 Fix return result for remote execution [ignite-3]

2024-11-18 Thread via GitHub
valepakh opened a new pull request, #4736: URL: https://github.com/apache/ignite-3/pull/4736 https://issues.apache.org/jira/browse/IGNITE-23696 Complete result future exceptionally when cancellation was requested before the job completion. -- This is an automated message from the A