[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #2143: IGNITE-19646 Transform IndexManager to internally work against Catalog event types

2023-06-07 Thread via GitHub
tkalkirill commented on code in PR #2143: URL: https://github.com/apache/ignite-3/pull/2143#discussion_r1221022352 ## modules/storage-rocksdb/src/main/java/org/apache/ignite/internal/storage/rocksdb/RocksDbTableStorage.java: ## @@ -765,4 +781,18 @@ private List getSortedIndexSt

[GitHub] [ignite-3] rpuch commented on a diff in pull request #2155: IGNITE-19648 Failed to cancel rebalance

2023-06-07 Thread via GitHub
rpuch commented on code in PR #2155: URL: https://github.com/apache/ignite-3/pull/2155#discussion_r1221006155 ## modules/storage-api/src/main/java/org/apache/ignite/internal/storage/util/StorageOperation.java: ## @@ -109,6 +109,34 @@ String inProcessErrorMessage(String storageIn

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #2155: IGNITE-19648 Failed to cancel rebalance

2023-06-07 Thread via GitHub
tkalkirill commented on code in PR #2155: URL: https://github.com/apache/ignite-3/pull/2155#discussion_r1221075045 ## modules/storage-api/src/main/java/org/apache/ignite/internal/storage/util/StorageOperation.java: ## @@ -109,6 +109,34 @@ String inProcessErrorMessage(String stor

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #2155: IGNITE-19648 Failed to cancel rebalance

2023-06-07 Thread via GitHub
tkalkirill commented on code in PR #2155: URL: https://github.com/apache/ignite-3/pull/2155#discussion_r1221076805 ## modules/storage-api/src/main/java/org/apache/ignite/internal/storage/util/MvPartitionStorages.java: ## @@ -415,7 +430,15 @@ private String createStorageInProgre

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #2155: IGNITE-19648 Failed to cancel rebalance

2023-06-07 Thread via GitHub
tkalkirill commented on code in PR #2155: URL: https://github.com/apache/ignite-3/pull/2155#discussion_r1221078830 ## modules/storage-api/src/main/java/org/apache/ignite/internal/storage/util/MvPartitionStorages.java: ## @@ -238,14 +240,16 @@ public CompletableFuture startRebala

[GitHub] [ignite-3] sashapolo commented on a diff in pull request #2143: IGNITE-19646 Transform IndexManager to internally work against Catalog event types

2023-06-07 Thread via GitHub
sashapolo commented on code in PR #2143: URL: https://github.com/apache/ignite-3/pull/2143#discussion_r1221082188 ## modules/index/src/main/java/org/apache/ignite/internal/index/IndexManager.java: ## @@ -436,31 +436,39 @@ private CompletableFuture onIndexCreate(ConfigurationNot

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #2155: IGNITE-19648 Failed to cancel rebalance

2023-06-07 Thread via GitHub
tkalkirill commented on code in PR #2155: URL: https://github.com/apache/ignite-3/pull/2155#discussion_r1221084037 ## modules/storage-api/src/test/java/org/apache/ignite/internal/storage/util/MvPartitionStoragesTest.java: ## @@ -326,8 +326,6 @@ void testStartRebalance() {

[GitHub] [ignite-3] sashapolo commented on a diff in pull request #2143: IGNITE-19646 Transform IndexManager to internally work against Catalog event types

2023-06-07 Thread via GitHub
sashapolo commented on code in PR #2143: URL: https://github.com/apache/ignite-3/pull/2143#discussion_r1221084523 ## modules/index/src/main/java/org/apache/ignite/internal/index/IndexManager.java: ## @@ -436,31 +436,39 @@ private CompletableFuture onIndexCreate(ConfigurationNot

[GitHub] [ignite-3] sashapolo commented on a diff in pull request #2143: IGNITE-19646 Transform IndexManager to internally work against Catalog event types

2023-06-07 Thread via GitHub
sashapolo commented on code in PR #2143: URL: https://github.com/apache/ignite-3/pull/2143#discussion_r1221086830 ## modules/index/src/main/java/org/apache/ignite/internal/index/IndexManager.java: ## @@ -608,4 +599,54 @@ public CompletableFuture onUpdate(ConfigurationNotificati

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #2155: IGNITE-19648 Failed to cancel rebalance

2023-06-07 Thread via GitHub
tkalkirill commented on code in PR #2155: URL: https://github.com/apache/ignite-3/pull/2155#discussion_r1221087363 ## modules/storage-api/src/main/java/org/apache/ignite/internal/storage/util/StorageOperation.java: ## @@ -109,6 +109,34 @@ String inProcessErrorMessage(String stor

[GitHub] [ignite-3] sashapolo commented on a diff in pull request #2143: IGNITE-19646 Transform IndexManager to internally work against Catalog event types

2023-06-07 Thread via GitHub
sashapolo commented on code in PR #2143: URL: https://github.com/apache/ignite-3/pull/2143#discussion_r1221088319 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ClusterPerClassIntegrationTest.java: ## @@ -526,7 +526,7 @@ protected static Map> w

[GitHub] [ignite-3] sashapolo opened a new pull request, #2158: IGNITE-19658 Fix NPE in ItMetaStorageServiceTest

2023-06-07 Thread via GitHub
sashapolo opened a new pull request, #2158: URL: https://github.com/apache/ignite-3/pull/2158 https://issues.apache.org/jira/browse/IGNITE-19658 -- 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

[GitHub] [ignite] timoninmaxim commented on a diff in pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
timoninmaxim commented on code in PR #10675: URL: https://github.com/apache/ignite/pull/10675#discussion_r1221112180 ## modules/core/src/main/java/org/apache/ignite/internal/management/api/CliPositionalSubcommands.java: ## @@ -0,0 +1,35 @@ +/* + * Licensed to the Apache Software

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #2155: IGNITE-19648 Failed to cancel rebalance

2023-06-07 Thread via GitHub
tkalkirill commented on code in PR #2155: URL: https://github.com/apache/ignite-3/pull/2155#discussion_r1221113719 ## modules/storage-api/src/main/java/org/apache/ignite/internal/storage/util/StorageOperation.java: ## @@ -109,6 +109,34 @@ String inProcessErrorMessage(String stor

[GitHub] [ignite-3] rpuch commented on a diff in pull request #2155: IGNITE-19648 Failed to cancel rebalance

2023-06-07 Thread via GitHub
rpuch commented on code in PR #2155: URL: https://github.com/apache/ignite-3/pull/2155#discussion_r1221124867 ## modules/storage-api/src/test/java/org/apache/ignite/internal/storage/util/MvPartitionStoragesTest.java: ## @@ -326,8 +326,6 @@ void testStartRebalance() {

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #2143: IGNITE-19646 Transform IndexManager to internally work against Catalog event types

2023-06-07 Thread via GitHub
tkalkirill commented on code in PR #2143: URL: https://github.com/apache/ignite-3/pull/2143#discussion_r1221125474 ## modules/index/src/main/java/org/apache/ignite/internal/index/IndexManager.java: ## @@ -436,31 +436,39 @@ private CompletableFuture onIndexCreate(ConfigurationNo

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #2143: IGNITE-19646 Transform IndexManager to internally work against Catalog event types

2023-06-07 Thread via GitHub
tkalkirill commented on code in PR #2143: URL: https://github.com/apache/ignite-3/pull/2143#discussion_r1221126692 ## modules/index/src/main/java/org/apache/ignite/internal/index/IndexManager.java: ## @@ -436,31 +436,39 @@ private CompletableFuture onIndexCreate(ConfigurationNo

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #2143: IGNITE-19646 Transform IndexManager to internally work against Catalog event types

2023-06-07 Thread via GitHub
tkalkirill commented on code in PR #2143: URL: https://github.com/apache/ignite-3/pull/2143#discussion_r1221128770 ## modules/index/src/main/java/org/apache/ignite/internal/index/IndexManager.java: ## @@ -608,4 +599,54 @@ public CompletableFuture onUpdate(ConfigurationNotificat

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #2143: IGNITE-19646 Transform IndexManager to internally work against Catalog event types

2023-06-07 Thread via GitHub
tkalkirill commented on code in PR #2143: URL: https://github.com/apache/ignite-3/pull/2143#discussion_r1221149497 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ClusterPerClassIntegrationTest.java: ## @@ -526,7 +526,7 @@ protected static Map>

[GitHub] [ignite-3] SammyVimes merged pull request #2158: IGNITE-19658 Fix NPE in ItMetaStorageServiceTest

2023-06-07 Thread via GitHub
SammyVimes merged PR #2158: URL: https://github.com/apache/ignite-3/pull/2158 -- 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...@

[GitHub] [ignite-3] tkalkirill merged pull request #2155: IGNITE-19648 Failed to cancel rebalance

2023-06-07 Thread via GitHub
tkalkirill merged PR #2155: URL: https://github.com/apache/ignite-3/pull/2155 -- 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...@

[GitHub] [ignite] nizhikov commented on a diff in pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
nizhikov commented on code in PR #10675: URL: https://github.com/apache/ignite/pull/10675#discussion_r1221196751 ## modules/core/src/main/java/org/apache/ignite/internal/management/api/Command.java: ## @@ -0,0 +1,65 @@ +/* + * Licensed to the Apache Software Foundation (ASF) und

[GitHub] [ignite] nizhikov commented on a diff in pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
nizhikov commented on code in PR #10675: URL: https://github.com/apache/ignite/pull/10675#discussion_r1221208755 ## modules/core/src/main/java/org/apache/ignite/internal/management/cache/CacheCommand.java: ## @@ -0,0 +1,59 @@ +/* + * Licensed to the Apache Software Foundation (A

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10675: URL: https://github.com/apache/ignite/pull/10675#issuecomment-1580269491 SonarCloud Quality Gate failed.    [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality Ga

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10675: URL: https://github.com/apache/ignite/pull/10675#issuecomment-1580285843 SonarCloud Quality Gate failed.    [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality Ga

[GitHub] [ignite-3] tkalkirill merged pull request #2143: IGNITE-19646 Transform IndexManager to internally work against Catalog event types

2023-06-07 Thread via GitHub
tkalkirill merged PR #2143: URL: https://github.com/apache/ignite-3/pull/2143 -- 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...@

[GitHub] [ignite-3] xtern merged pull request #2105: IGNITE-19407 Sql. Introduce DDL command for ALTER COLUMN TYPE.

2023-06-07 Thread via GitHub
xtern merged PR #2105: URL: https://github.com/apache/ignite-3/pull/2105 -- 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

[GitHub] [ignite-3] zstan opened a new pull request, #2159: IGNITE-19341 Sql. Correctly process null`s in SUBSTRING

2023-06-07 Thread via GitHub
zstan opened a new pull request, #2159: URL: https://github.com/apache/ignite-3/pull/2159 (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-mai

[GitHub] [ignite-3] rpuch commented on a diff in pull request #2154: IGNITE-19642 CatalogService should use HybridClock to register new Catalog version

2023-06-07 Thread via GitHub
rpuch commented on code in PR #2154: URL: https://github.com/apache/ignite-3/pull/2154#discussion_r1221353399 ## modules/catalog/src/main/java/org/apache/ignite/internal/catalog/CatalogServiceImpl.java: ## @@ -519,7 +536,7 @@ public void handle(VersionedUpdate update) {

[GitHub] [ignite] alex-plekhanov commented on a diff in pull request #10768: IGNITE-19650 Fixed broken serialization of communication messages due to incorrect GridCacheQueryRequest class marshalling.

2023-06-07 Thread via GitHub
alex-plekhanov commented on code in PR #10768: URL: https://github.com/apache/ignite/pull/10768#discussion_r1221414869 ## modules/core/src/test/java/org/apache/ignite/internal/managers/communication/IgniteMessageFactoryImplTest.java: ## @@ -112,6 +132,75 @@ public void testRegis

[GitHub] [ignite-3] ibessonov opened a new pull request, #2161: IGNITE-19422 Fixed "get" method in index storages.

2023-06-07 Thread via GitHub
ibessonov opened a new pull request, #2161: URL: https://github.com/apache/ignite-3/pull/2161 https://issues.apache.org/jira/browse/IGNITE-19422 -- 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

[GitHub] [ignite-3] korlov42 commented on a diff in pull request #2047: IGNITE-18831 Sql. Fix dynamic parameters type coercion.

2023-06-07 Thread via GitHub
korlov42 commented on code in PR #2047: URL: https://github.com/apache/ignite-3/pull/2047#discussion_r1221528338 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/prepare/IgniteSqlValidator.java: ## @@ -306,70 +358,46 @@ public void validateAggregateParam

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10732: IGNITE-19529 IgniteTxStateImpl's non-thread safe fields may cause incorrect node behavior

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10732: URL: https://github.com/apache/ignite/pull/10732#issuecomment-1580771425 SonarCloud Quality Gate failed.    [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality Ga

[GitHub] [ignite-3] zstan opened a new pull request, #2162: IGNITE-19493 Sql. Change query execution flow

2023-06-07 Thread via GitHub
zstan opened a new pull request, #2162: URL: https://github.com/apache/ignite-3/pull/2162 (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-mai

[GitHub] [ignite] nizhikov commented on a diff in pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
nizhikov commented on code in PR #10675: URL: https://github.com/apache/ignite/pull/10675#discussion_r1221561302 ## modules/core/src/main/java/org/apache/ignite/internal/management/api/WithCliConfirmParameter.java: ## @@ -0,0 +1,33 @@ +/* + * Licensed to the Apache Software Foun

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10675: URL: https://github.com/apache/ignite/pull/10675#issuecomment-1580789064 SonarCloud Quality Gate failed.    [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality Ga

[GitHub] [ignite-3] AMashenkov commented on pull request #2154: IGNITE-19642 CatalogService should use HybridClock to register new Catalog version

2023-06-07 Thread via GitHub
AMashenkov commented on PR #2154: URL: https://github.com/apache/ignite-3/pull/2154#issuecomment-1580800975 > Another question is about `CatalogService` API: it's still using `long` to accept/pass timestamps, not `HybridTimestamp`. Is it planned to change this? Is it intentional that in the

[GitHub] [ignite-3] korlov42 commented on a diff in pull request #2047: IGNITE-18831 Sql. Fix dynamic parameters type coercion.

2023-06-07 Thread via GitHub
korlov42 commented on code in PR #2047: URL: https://github.com/apache/ignite-3/pull/2047#discussion_r1221592072 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/prepare/IgniteTypeCoercion.java: ## @@ -392,13 +587,135 @@ private static SqlNode castTo(Sql

[GitHub] [ignite] nizhikov commented on a diff in pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
nizhikov commented on code in PR #10675: URL: https://github.com/apache/ignite/pull/10675#discussion_r1221598570 ## modules/core/src/main/java/org/apache/ignite/internal/management/api/CliPositionalSubcommands.java: ## @@ -0,0 +1,35 @@ +/* + * Licensed to the Apache Software Fou

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2162: IGNITE-19493 Sql. Change query execution flow

2023-06-07 Thread via GitHub
AMashenkov commented on code in PR #2162: URL: https://github.com/apache/ignite-3/pull/2162#discussion_r1221602608 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/SqlQueryProcessor.java: ## @@ -425,6 +421,18 @@ private CompletableFuture>> querySingle0(

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2162: IGNITE-19493 Sql. Change query execution flow

2023-06-07 Thread via GitHub
AMashenkov commented on code in PR #2162: URL: https://github.com/apache/ignite-3/pull/2162#discussion_r1221607145 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/SqlQueryProcessor.java: ## @@ -425,6 +421,18 @@ private CompletableFuture>> querySingle0(

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2162: IGNITE-19493 Sql. Change query execution flow

2023-06-07 Thread via GitHub
AMashenkov commented on code in PR #2162: URL: https://github.com/apache/ignite-3/pull/2162#discussion_r1221608178 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/SqlQueryProcessor.java: ## @@ -425,6 +421,18 @@ private CompletableFuture>> querySingle0(

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2162: IGNITE-19493 Sql. Change query execution flow

2023-06-07 Thread via GitHub
AMashenkov commented on code in PR #2162: URL: https://github.com/apache/ignite-3/pull/2162#discussion_r122160 ## modules/runner/src/integrationTest/sql/types/timestamp/test_incorrect_timestamp.test: ## @@ -1,7 +1,6 @@ # name: test/sql/types/timestamp/test_incorrect_timesta

[GitHub] [ignite-3] zstan commented on a diff in pull request #2162: IGNITE-19493 Sql. Change query execution flow

2023-06-07 Thread via GitHub
zstan commented on code in PR #2162: URL: https://github.com/apache/ignite-3/pull/2162#discussion_r1221610186 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/SqlQueryProcessor.java: ## @@ -425,6 +421,18 @@ private CompletableFuture>> querySingle0(

[GitHub] [ignite-3] zstan commented on a diff in pull request #2162: IGNITE-19493 Sql. Change query execution flow

2023-06-07 Thread via GitHub
zstan commented on code in PR #2162: URL: https://github.com/apache/ignite-3/pull/2162#discussion_r1221611264 ## modules/runner/src/integrationTest/sql/types/timestamp/test_incorrect_timestamp.test: ## @@ -1,7 +1,6 @@ # name: test/sql/types/timestamp/test_incorrect_timestamp.te

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10675: URL: https://github.com/apache/ignite/pull/10675#issuecomment-1580833912 SonarCloud Quality Gate failed.    [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality Ga

[GitHub] [ignite-3] zstan commented on a diff in pull request #2162: IGNITE-19493 Sql. Change query execution flow

2023-06-07 Thread via GitHub
zstan commented on code in PR #2162: URL: https://github.com/apache/ignite-3/pull/2162#discussion_r1221628218 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/SqlQueryProcessor.java: ## @@ -425,6 +421,18 @@ private CompletableFuture>> querySingle0(

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10768: IGNITE-19650 Fixed broken serialization of communication messages due to incorrect GridCacheQueryRequest class marshalling.

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10768: URL: https://github.com/apache/ignite/pull/10768#issuecomment-1580855044 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Qua

[GitHub] [ignite] timoninmaxim commented on a diff in pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
timoninmaxim commented on code in PR #10675: URL: https://github.com/apache/ignite/pull/10675#discussion_r1221630268 ## modules/core/src/main/java/org/apache/ignite/internal/management/api/Command.java: ## @@ -0,0 +1,69 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10675: URL: https://github.com/apache/ignite/pull/10675#issuecomment-1580877059 SonarCloud Quality Gate failed.    [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality Ga

[GitHub] [ignite-3] sashapolo opened a new pull request, #2163: IGNITE-19451 Prohibit some types from being Marshallable

2023-06-07 Thread via GitHub
sashapolo opened a new pull request, #2163: URL: https://github.com/apache/ignite-3/pull/2163 https://issues.apache.org/jira/browse/IGNITE-19451 -- 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

[GitHub] [ignite-3] sanpwc commented on a diff in pull request #2142: IGNITE-19494 Correctly stop a replica

2023-06-07 Thread via GitHub
sanpwc commented on code in PR #2142: URL: https://github.com/apache/ignite-3/pull/2142#discussion_r1221656137 ## modules/raft/src/main/java/org/apache/ignite/raft/jraft/disruptor/StripedDisruptor.java: ## @@ -214,7 +214,7 @@ void unsubscribe(NodeId nodeId) { assert

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2159: IGNITE-19341 Sql. Correctly process null`s in SUBSTRING

2023-06-07 Thread via GitHub
AMashenkov commented on code in PR #2159: URL: https://github.com/apache/ignite-3/pull/2159#discussion_r1221658256 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItFunctionsTest.java: ## @@ -285,16 +285,37 @@ public void testTypeOf() {

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2159: IGNITE-19341 Sql. Correctly process null`s in SUBSTRING

2023-06-07 Thread via GitHub
AMashenkov commented on code in PR #2159: URL: https://github.com/apache/ignite-3/pull/2159#discussion_r1221658256 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItFunctionsTest.java: ## @@ -285,16 +285,37 @@ public void testTypeOf() {

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2056: IGNITE-15507: correct type mapping for JDBC metadata.

2023-06-07 Thread via GitHub
AMashenkov commented on code in PR #2056: URL: https://github.com/apache/ignite-3/pull/2056#discussion_r1211722780 ## modules/api/src/main/java/org/apache/ignite/sql/ColumnType.java: ## @@ -164,4 +164,23 @@ public static Class columnTypeToClass(ColumnType type) {

[GitHub] [ignite-3] vldpyatkov merged pull request #2153: IGNITE-19457 Review nullability of network messages' fields

2023-06-07 Thread via GitHub
vldpyatkov merged PR #2153: URL: https://github.com/apache/ignite-3/pull/2153 -- 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...@

[GitHub] [ignite-3] sanpwc commented on a diff in pull request #2142: IGNITE-19494 Correctly stop a replica

2023-06-07 Thread via GitHub
sanpwc commented on code in PR #2142: URL: https://github.com/apache/ignite-3/pull/2142#discussion_r1221695867 ## modules/replicator/src/main/java/org/apache/ignite/internal/replicator/exception/ReplicaStoppingException.java: ## @@ -0,0 +1,51 @@ +/* + * Licensed to the Apache So

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10675: URL: https://github.com/apache/ignite/pull/10675#issuecomment-1580947874 SonarCloud Quality Gate failed.    [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality Ga

[GitHub] [ignite-3] ibessonov commented on a diff in pull request #2150: IGNITE-19657 Remove schema from BinaryTuple implementation

2023-06-07 Thread via GitHub
ibessonov commented on code in PR #2150: URL: https://github.com/apache/ignite-3/pull/2150#discussion_r1221549641 ## modules/schema/src/main/java/org/apache/ignite/internal/schema/BinaryTuplePrefix.java: ## @@ -75,17 +71,17 @@ public static BinaryTuplePrefix fromBinaryTuple(Bina

[GitHub] [ignite] timoninmaxim commented on a diff in pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
timoninmaxim commented on code in PR #10675: URL: https://github.com/apache/ignite/pull/10675#discussion_r1221733917 ## modules/control-utility/src/main/java/org/apache/ignite/internal/commandline/CommandInvoker.java: ## @@ -0,0 +1,323 @@ +/* + * Licensed to the Apache Software

[GitHub] [ignite-3] Pochatkin commented on a diff in pull request #2121: IGNITE-19565 Use JobClassLoader for loading job classes

2023-06-07 Thread via GitHub
Pochatkin commented on code in PR #2121: URL: https://github.com/apache/ignite-3/pull/2121#discussion_r1221710843 ## modules/api/src/main/java/org/apache/ignite/compute/DeploymentUnit.java: ## @@ -41,6 +41,17 @@ public DeploymentUnit(String name, Version version) { this

[GitHub] [ignite] nizhikov commented on a diff in pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
nizhikov commented on code in PR #10675: URL: https://github.com/apache/ignite/pull/10675#discussion_r1221801192 ## modules/core/src/main/java/org/apache/ignite/internal/management/api/Command.java: ## @@ -0,0 +1,64 @@ +/* + * Licensed to the Apache Software Foundation (ASF) und

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10675: URL: https://github.com/apache/ignite/pull/10675#issuecomment-1581068182 SonarCloud Quality Gate failed.    [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality Ga

[GitHub] [ignite] NSAmelchev commented on a diff in pull request #10764: IGNITE-19652 Adapt to DirectByteBuffer ctor changes in JDK 21

2023-06-07 Thread via GitHub
NSAmelchev commented on code in PR #10764: URL: https://github.com/apache/ignite/pull/10764#discussion_r1221802767 ## modules/core/src/main/java/org/apache/ignite/internal/util/GridUnsafe.java: ## @@ -1660,6 +1661,12 @@ private static Method newDirectBufferMethod(Object nioAcce

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10732: IGNITE-19529 IgniteTxStateImpl's non-thread safe fields may cause incorrect node behavior

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10732: URL: https://github.com/apache/ignite/pull/10732#issuecomment-1581074744 SonarCloud Quality Gate failed.    [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality Ga

[GitHub] [ignite] NSAmelchev commented on pull request #10764: IGNITE-19652 Adapt to DirectByteBuffer ctor changes in JDK 21

2023-06-07 Thread via GitHub
NSAmelchev commented on PR #10764: URL: https://github.com/apache/ignite/pull/10764#issuecomment-1581078508 Hello, @wendigo. Please, fix the codestyle. I suggest writing to the dev list to ask for help with running CI on a new JDK. -- This is an automated message from the Apache Git

[GitHub] [ignite] nizhikov commented on a diff in pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
nizhikov commented on code in PR #10675: URL: https://github.com/apache/ignite/pull/10675#discussion_r1221810571 ## modules/control-utility/src/main/java/org/apache/ignite/internal/commandline/CommandInvoker.java: ## @@ -0,0 +1,323 @@ +/* + * Licensed to the Apache Software Foun

[GitHub] [ignite] nizhikov commented on a diff in pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
nizhikov commented on code in PR #10675: URL: https://github.com/apache/ignite/pull/10675#discussion_r1221810925 ## modules/core/src/main/java/org/apache/ignite/internal/management/api/ComputeCommand.java: ## @@ -0,0 +1,55 @@ +/* + * Licensed to the Apache Software Foundation (A

[GitHub] [ignite-3] PakhomovAlexander merged pull request #2156: IGNITE-19659 NPE in ClusterConfigRegistryImpl.fetchConfig

2023-06-07 Thread via GitHub
PakhomovAlexander merged PR #2156: URL: https://github.com/apache/ignite-3/pull/2156 -- 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-unsub

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10675: URL: https://github.com/apache/ignite/pull/10675#issuecomment-1581095464 SonarCloud Quality Gate failed.    [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality Ga

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10675: URL: https://github.com/apache/ignite/pull/10675#issuecomment-1581122153 SonarCloud Quality Gate failed.    [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality Ga

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10675: URL: https://github.com/apache/ignite/pull/10675#issuecomment-1581139930 SonarCloud Quality Gate failed.    [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality Ga

[GitHub] [ignite-3] xtern opened a new pull request, #2164: IGNITE-17765

2023-06-07 Thread via GitHub
xtern opened a new pull request, #2164: URL: https://github.com/apache/ignite-3/pull/2164 (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-mai

[GitHub] [ignite] petrov-mg merged pull request #10768: IGNITE-19650 Fixed broken serialization of communication messages due to incorrect GridCacheQueryRequest class marshalling.

2023-06-07 Thread via GitHub
petrov-mg merged PR #10768: URL: https://github.com/apache/ignite/pull/10768 -- 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

[GitHub] [ignite] yurinaryshkin commented on a diff in pull request #10767: IGNITE-16619 IndexQuery should support limit

2023-06-07 Thread via GitHub
yurinaryshkin commented on code in PR #10767: URL: https://github.com/apache/ignite/pull/10767#discussion_r1221975526 ## modules/core/src/main/java/org/apache/ignite/cache/query/IndexQuery.java: ## @@ -94,11 +97,34 @@ public IndexQuery(Class valCls, @Nullable String idxName) {

[GitHub] [ignite] yurinaryshkin commented on a diff in pull request #10767: IGNITE-16619 IndexQuery should support limit

2023-06-07 Thread via GitHub
yurinaryshkin commented on code in PR #10767: URL: https://github.com/apache/ignite/pull/10767#discussion_r1221976007 ## modules/indexing/src/test/java/org/apache/ignite/cache/query/IndexQueryLimitTest.java: ## @@ -0,0 +1,350 @@ +package org.apache.ignite.cache.query; + +import

[GitHub] [ignite] yurinaryshkin commented on a diff in pull request #10767: IGNITE-16619 IndexQuery should support limit

2023-06-07 Thread via GitHub
yurinaryshkin commented on code in PR #10767: URL: https://github.com/apache/ignite/pull/10767#discussion_r1221977577 ## modules/indexing/src/test/java/org/apache/ignite/cache/query/IndexQueryTestSuite.java: ## @@ -46,7 +46,8 @@ ThinClientIndexQueryTest.class, Review Comme

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10675: URL: https://github.com/apache/ignite/pull/10675#issuecomment-1581270273 SonarCloud Quality Gate failed.    [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality Ga

[GitHub] [ignite] yurinaryshkin commented on a diff in pull request #10767: IGNITE-16619 IndexQuery should support limit

2023-06-07 Thread via GitHub
yurinaryshkin commented on code in PR #10767: URL: https://github.com/apache/ignite/pull/10767#discussion_r1221978451 ## modules/core/src/main/java/org/apache/ignite/cache/query/IndexQuery.java: ## @@ -152,6 +178,27 @@ public String getIndexName() { return idxName;

[GitHub] [ignite-3] alievmirza opened a new pull request, #2165: IGNITE-19674 introduce TestRocksDbKeyValueStorage

2023-06-07 Thread via GitHub
alievmirza opened a new pull request, #2165: URL: https://github.com/apache/ignite-3/pull/2165 https://issues.apache.org/jira/browse/IGNITE-19674 -- 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

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10767: IGNITE-16619 IndexQuery should support limit

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10767: URL: https://github.com/apache/ignite/pull/10767#issuecomment-1581298235 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Qua

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10767: IGNITE-16619 IndexQuery should support limit

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10767: URL: https://github.com/apache/ignite/pull/10767#issuecomment-1581327555 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Qua

[GitHub] [ignite] timoninmaxim commented on a diff in pull request #10767: IGNITE-16619 IndexQuery should support limit

2023-06-07 Thread via GitHub
timoninmaxim commented on code in PR #10767: URL: https://github.com/apache/ignite/pull/10767#discussion_r1222000811 ## modules/core/src/main/java/org/apache/ignite/internal/processors/platform/client/cache/ClientCacheIndexQueryRequest.java: ## @@ -57,6 +57,8 @@ public ClientCac

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10732: IGNITE-19529 IgniteTxStateImpl's non-thread safe fields may cause incorrect node behavior

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10732: URL: https://github.com/apache/ignite/pull/10732#issuecomment-1581350975 SonarCloud Quality Gate failed.    [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality Ga

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10764: IGNITE-19652 Adapt to DirectByteBuffer ctor changes in JDK 21

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10764: URL: https://github.com/apache/ignite/pull/10764#issuecomment-1581490608 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Qua

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10764: IGNITE-19652 Adapt to DirectByteBuffer ctor changes in JDK 21

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10764: URL: https://github.com/apache/ignite/pull/10764#issuecomment-1581506634 Kudos, SonarCloud Quality Gate passed!    [![Quality Gate passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png 'Qua

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10675: IGNITE-15629 Management API introduced

2023-06-07 Thread via GitHub
sonarcloud[bot] commented on PR #10675: URL: https://github.com/apache/ignite/pull/10675#issuecomment-1581531798 SonarCloud Quality Gate failed.    [![Quality Gate failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/failed-16px.png 'Quality Ga

[GitHub] [ignite-3] sashapolo opened a new pull request, #2166: IGNITE-19678 Fix remaining NPEs in ItMetaStorageServiceTest

2023-06-07 Thread via GitHub
sashapolo opened a new pull request, #2166: URL: https://github.com/apache/ignite-3/pull/2166 https://issues.apache.org/jira/browse/IGNITE-19678 -- 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

[GitHub] [ignite-3] sashapolo commented on a diff in pull request #2150: IGNITE-19657 Remove schema from BinaryTuple implementation

2023-06-07 Thread via GitHub
sashapolo commented on code in PR #2150: URL: https://github.com/apache/ignite-3/pull/2150#discussion_r1222493666 ## modules/schema/src/main/java/org/apache/ignite/internal/schema/BinaryTuplePrefix.java: ## @@ -75,17 +71,17 @@ public static BinaryTuplePrefix fromBinaryTuple(Bina

[GitHub] [ignite-3] sashapolo commented on a diff in pull request #2150: IGNITE-19657 Remove schema from BinaryTuple implementation

2023-06-07 Thread via GitHub
sashapolo commented on code in PR #2150: URL: https://github.com/apache/ignite-3/pull/2150#discussion_r1222494996 ## modules/schema/src/main/java/org/apache/ignite/internal/schema/BinaryTuple.java: ## @@ -27,59 +27,33 @@ * elements as objects. */ public class BinaryTuple ex

[GitHub] [ignite-3] sashapolo commented on a diff in pull request #2150: IGNITE-19657 Remove schema from BinaryTuple implementation

2023-06-07 Thread via GitHub
sashapolo commented on code in PR #2150: URL: https://github.com/apache/ignite-3/pull/2150#discussion_r1222515812 ## modules/schema/src/main/java/org/apache/ignite/internal/schema/BinaryTuplePrefix.java: ## @@ -75,17 +71,17 @@ public static BinaryTuplePrefix fromBinaryTuple(Bina

[GitHub] [ignite-3] sashapolo commented on a diff in pull request #2150: IGNITE-19657 Remove schema from BinaryTuple implementation

2023-06-07 Thread via GitHub
sashapolo commented on code in PR #2150: URL: https://github.com/apache/ignite-3/pull/2150#discussion_r1222516479 ## modules/schema/src/main/java/org/apache/ignite/internal/schema/BinaryTuple.java: ## @@ -27,59 +27,33 @@ * elements as objects. */ public class BinaryTuple ex

[GitHub] [ignite-3] ptupitsyn merged pull request #2166: IGNITE-19678 Fix remaining NPEs in ItMetaStorageServiceTest

2023-06-07 Thread via GitHub
ptupitsyn merged PR #2166: URL: https://github.com/apache/ignite-3/pull/2166 -- 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