[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #1562: IGNITE-18030 Integration of the new full rebalance API with IncomingSnapshotCopier

tkalkirill commented on code in PR #1562: URL: https://github.com/apache/ignite-3/pull/1562#discussion_r1084898716 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/raft/snapshot/incoming/IncomingSnapshotCopier.java: ## @@ -96,38 +102,37 @@ public

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #1562: IGNITE-18030 Integration of the new full rebalance API with IncomingSnapshotCopier

tkalkirill commented on code in PR #1562: URL: https://github.com/apache/ignite-3/pull/1562#discussion_r1084898581 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/raft/snapshot/incoming/IncomingSnapshotCopier.java: ## @@ -157,9 +162,9 @@ public void

[GitHub] [ignite-3] rpuch commented on a diff in pull request #1565: IGNITE-18086 Add index building in IncomingSnapshotCopier

rpuch commented on code in PR #1565: URL: https://github.com/apache/ignite-3/pull/1565#discussion_r1084880005 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/raft/snapshot/PartitionAccessImpl.java: ## @@ -213,4 +229,8 @@ private TxStateStorage

[GitHub] [ignite-3] ibessonov commented on a diff in pull request #1562: IGNITE-18030 Integration of the new full rebalance API with IncomingSnapshotCopier

ibessonov commented on code in PR #1562: URL: https://github.com/apache/ignite-3/pull/1562#discussion_r1084876619 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/raft/snapshot/incoming/IncomingSnapshotCopier.java: ## @@ -96,38 +102,37 @@ public

[GitHub] [ignite-3] zstan commented on a diff in pull request #1553: IGNITE-18580 Sql. Redesign the Exchange to use a pull-based approach

zstan commented on code in PR #1553: URL: https://github.com/apache/ignite-3/pull/1553#discussion_r1083696938 ## modules/sql-engine/src/test/java/org/apache/ignite/internal/sql/engine/exec/rel/ExchangeExecutionTest.java: ## @@ -0,0 +1,202 @@ +/* + * Licensed to the Apache

[GitHub] [ignite-3] zstan merged pull request #1550: IGNITE-18585 Sql. Introduce cache for serialized RelNodes representation

zstan merged PR #1550: URL: https://github.com/apache/ignite-3/pull/1550 -- 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:

[GitHub] [ignite-web-console] dependabot[bot] opened a new pull request, #23: Bump cookiejar from 2.1.2 to 2.1.4 in /modules/backend

dependabot[bot] opened a new pull request, #23: URL: https://github.com/apache/ignite-web-console/pull/23 Bumps [cookiejar](https://github.com/bmeck/node-cookiejar) from 2.1.2 to 2.1.4. Commits See full diff in https://github.com/bmeck/node-cookiejar/commits;>compare view

[GitHub] [ignite] nizhikov opened a new pull request, #10492: IGNITE-18550 Incremental snapshot verify command

nizhikov opened a new pull request, #10492: URL: https://github.com/apache/ignite/pull/10492 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: ###

[GitHub] [ignite-3] tkalkirill opened a new pull request, #1565: IGNITE-18086 Add index building in IncomingSnapshotCopier

tkalkirill opened a new pull request, #1565: URL: https://github.com/apache/ignite-3/pull/1565 https://issues.apache.org/jira/browse/IGNITE-18086 -- 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

[GitHub] [ignite-3] vldpyatkov commented on a diff in pull request #1551: IGNITE-18527

vldpyatkov commented on code in PR #1551: URL: https://github.com/apache/ignite-3/pull/1551#discussion_r1084200437 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/replicator/PartitionReplicaListener.java: ## @@ -1388,6 +1418,62 @@ private

[GitHub] [ignite-3] sanpwc merged pull request #1561: IGNITE-18600 Save Meta Storage watch data to the Vault

sanpwc merged PR #1561: URL: https://github.com/apache/ignite-3/pull/1561 -- 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:

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #1556: IGNITE-18593 Get rid of MvPartitionStorage and TxStateStorage in PartitionAccess

tkalkirill commented on code in PR #1556: URL: https://github.com/apache/ignite-3/pull/1556#discussion_r1083916784 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/raft/snapshot/PartitionAccess.java: ## @@ -32,27 +41,111 @@ public interface

[GitHub] [ignite] meanother commented on issue #10491: CorruptedTreeException: B+Tree is corrupted

meanother commented on issue #10491: URL: https://github.com/apache/ignite/issues/10491#issuecomment-1400429078 1) Create table `X`, fill it with data (or they are already there) 2) Create table saa_sv_rawdbo_dm_transactions_month, create index idx_fin_client (both entities are empty)

[GitHub] [ignite] ivandasch commented on issue #10491: CorruptedTreeException: B+Tree is corrupted

ivandasch commented on issue #10491: URL: https://github.com/apache/ignite/issues/10491#issuecomment-1400415252 @meanother Do I understand correctly, that you have tried to create an additional index on the key field and this action led to a failure? -- This is an automated message from

[GitHub] [ignite] meanother opened a new issue, #10491: CorruptedTreeException: B+Tree is corrupted

meanother opened a new issue, #10491: URL: https://github.com/apache/ignite/issues/10491 Java version 1.8 Ignite version 2.14 ``` CREATE TABLE saa_sv_rawdbo_dm_transactions_month( pay_client_pin VARCHAR , pay_transaction_count int , pay_tarif_plan_set VARCHAR

[GitHub] [ignite] nizhikov merged pull request #10485: IGNITE-18232 Remove of CacheMetricsMXBean legacy JMX bean

nizhikov merged PR #10485: URL: https://github.com/apache/ignite/pull/10485 -- 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:

[GitHub] [ignite-3] tkalkirill merged pull request #1556: IGNITE-18593 Get rid of MvPartitionStorage and TxStateStorage in PartitionAccess

tkalkirill merged PR #1556: URL: https://github.com/apache/ignite-3/pull/1556 -- 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:

[GitHub] [ignite] timoninmaxim merged pull request #10353: IGNITE-17648 Extract CacheStripedExecutor to separate class

timoninmaxim merged PR #10353: URL: https://github.com/apache/ignite/pull/10353 -- 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:

[GitHub] [ignite] alex-plekhanov commented on a diff in pull request #10479: IGNITE-18511 Added details to Calcite parser exception

alex-plekhanov commented on code in PR #10479: URL: https://github.com/apache/ignite/pull/10479#discussion_r1083965015 ## modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/integration/TableDdlIntegrationTest.java: ## @@ -281,6 +281,24 @@ public

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #1556: IGNITE-18593 Get rid of MvPartitionStorage and TxStateStorage in PartitionAccess

tkalkirill commented on code in PR #1556: URL: https://github.com/apache/ignite-3/pull/1556#discussion_r1083946905 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/raft/snapshot/PartitionAccessImpl.java: ## @@ -57,48 +70,147 @@ public PartitionKey

[GitHub] [ignite] Vladsz83 commented on a diff in pull request #10485: IGNITE-18232 : [IEP-80] Remove of CacheMetricsMXBean legacy JMX bean, v3 : keep JCache beans

Vladsz83 commented on code in PR #10485: URL: https://github.com/apache/ignite/pull/10485#discussion_r1083958072 ## modules/clients/src/test/java/org/apache/ignite/internal/processors/rest/RestMemcacheProtocolSelfTest.java: ## @@ -185,8 +185,8 @@ public void testReplace()

[GitHub] [ignite] nizhikov commented on a diff in pull request #10485: IGNITE-18232 : [IEP-80] Remove of CacheMetricsMXBean legacy JMX bean, v3 : keep JCache beans

nizhikov commented on code in PR #10485: URL: https://github.com/apache/ignite/pull/10485#discussion_r1083956343 ## modules/clients/src/test/java/org/apache/ignite/internal/processors/rest/RestMemcacheProtocolSelfTest.java: ## @@ -185,8 +185,8 @@ public void testReplace()

[GitHub] [ignite-3] rpuch commented on a diff in pull request #1556: IGNITE-18593 Get rid of MvPartitionStorage and TxStateStorage in PartitionAccess

rpuch commented on code in PR #1556: URL: https://github.com/apache/ignite-3/pull/1556#discussion_r1083937026 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/raft/snapshot/PartitionAccessImpl.java: ## @@ -57,48 +70,147 @@ public PartitionKey

[GitHub] [ignite] Vladsz83 commented on a diff in pull request #10485: IGNITE-18232 : [IEP-80] Remove of CacheMetricsMXBean legacy JMX bean, v3 : keep JCache beans

Vladsz83 commented on code in PR #10485: URL: https://github.com/apache/ignite/pull/10485#discussion_r1083943685 ## modules/clients/src/test/java/org/apache/ignite/internal/processors/rest/RestMemcacheProtocolSelfTest.java: ## @@ -185,8 +185,8 @@ public void testReplace()

[GitHub] [ignite] Vladsz83 commented on a diff in pull request #10485: IGNITE-18232 : [IEP-80] Remove of CacheMetricsMXBean legacy JMX bean, v3 : keep JCache beans

Vladsz83 commented on code in PR #10485: URL: https://github.com/apache/ignite/pull/10485#discussion_r1083943685 ## modules/clients/src/test/java/org/apache/ignite/internal/processors/rest/RestMemcacheProtocolSelfTest.java: ## @@ -185,8 +185,8 @@ public void testReplace()

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #1556: IGNITE-18593 Get rid of MvPartitionStorage and TxStateStorage in PartitionAccess

tkalkirill commented on code in PR #1556: URL: https://github.com/apache/ignite-3/pull/1556#discussion_r1083941400 ## modules/table/src/test/java/org/apache/ignite/internal/table/distributed/raft/snapshot/outgoing/OutgoingSnapshotCommonTest.java: ## @@ -76,12 +64,13 @@ void

[GitHub] [ignite-3] vldpyatkov opened a new pull request, #1563: IGNITE-18358 IGN-TX-5 on concurrent transactional single key load

vldpyatkov opened a new pull request, #1563: URL: https://github.com/apache/ignite-3/pull/1563 https://issues.apache.org/jira/browse/IGNITE-18358 -- 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

[GitHub] [ignite-3] korlov42 commented on a diff in pull request #1524: IGNITE-18464 Sql. Colocated sort aggregates need to compose a plans with additional sort

korlov42 commented on code in PR #1524: URL: https://github.com/apache/ignite-3/pull/1524#discussion_r1083927134 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/rel/SortAggregateNode.java: ## @@ -159,6 +161,10 @@ public void end() throws Exception

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #1556: IGNITE-18593 Get rid of MvPartitionStorage and TxStateStorage in PartitionAccess

tkalkirill commented on code in PR #1556: URL: https://github.com/apache/ignite-3/pull/1556#discussion_r1083924403 ## modules/table/src/test/java/org/apache/ignite/internal/table/distributed/raft/snapshot/PartitionSnapshotStorageFactoryTest.java: ## @@ -44,24 +39,18 @@ */

[GitHub] [ignite-3] rpuch commented on a diff in pull request #1556: IGNITE-18593 Get rid of MvPartitionStorage and TxStateStorage in PartitionAccess

rpuch commented on code in PR #1556: URL: https://github.com/apache/ignite-3/pull/1556#discussion_r1083932748 ## modules/table/src/test/java/org/apache/ignite/internal/table/distributed/raft/snapshot/outgoing/OutgoingSnapshotCommonTest.java: ## @@ -76,12 +64,13 @@ void

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #1556: IGNITE-18593 Get rid of MvPartitionStorage and TxStateStorage in PartitionAccess

tkalkirill commented on code in PR #1556: URL: https://github.com/apache/ignite-3/pull/1556#discussion_r1083926252 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/raft/snapshot/PartitionAccessImpl.java: ## @@ -57,48 +70,147 @@ public PartitionKey

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #1556: IGNITE-18593 Get rid of MvPartitionStorage and TxStateStorage in PartitionAccess

tkalkirill commented on code in PR #1556: URL: https://github.com/apache/ignite-3/pull/1556#discussion_r1083926252 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/raft/snapshot/PartitionAccessImpl.java: ## @@ -57,48 +70,147 @@ public PartitionKey

[GitHub] [ignite-3] korlov42 commented on a diff in pull request #1524: IGNITE-18464 Sql. Colocated sort aggregates need to compose a plans with additional sort

korlov42 commented on code in PR #1524: URL: https://github.com/apache/ignite-3/pull/1524#discussion_r1083924778 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItSortAggregateTest.java: ## @@ -67,8 +70,134 @@ public void mapReduceAggregate() {

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #1556: IGNITE-18593 Get rid of MvPartitionStorage and TxStateStorage in PartitionAccess

tkalkirill commented on code in PR #1556: URL: https://github.com/apache/ignite-3/pull/1556#discussion_r1083921930 ## modules/table/src/test/java/org/apache/ignite/internal/table/distributed/raft/snapshot/outgoing/OutgoingSnapshotCommonTest.java: ## @@ -76,12 +64,13 @@ void

[GitHub] [ignite] nizhikov commented on a diff in pull request #10485: IGNITE-18232 : [IEP-80] Remove of CacheMetricsMXBean legacy JMX bean, v3 : keep JCache beans

nizhikov commented on code in PR #10485: URL: https://github.com/apache/ignite/pull/10485#discussion_r1083923917 ## modules/clients/src/test/java/org/apache/ignite/internal/processors/rest/RestMemcacheProtocolSelfTest.java: ## @@ -185,8 +185,8 @@ public void testReplace()

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #1556: IGNITE-18593 Get rid of MvPartitionStorage and TxStateStorage in PartitionAccess

tkalkirill commented on code in PR #1556: URL: https://github.com/apache/ignite-3/pull/1556#discussion_r1083916394 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/raft/snapshot/PartitionAccess.java: ## @@ -32,27 +41,111 @@ public interface

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #1556: IGNITE-18593 Get rid of MvPartitionStorage and TxStateStorage in PartitionAccess

tkalkirill commented on code in PR #1556: URL: https://github.com/apache/ignite-3/pull/1556#discussion_r1083918501 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/raft/snapshot/incoming/IncomingSnapshotCopier.java: ## @@ -186,32 +186,13 @@ public

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #1556: IGNITE-18593 Get rid of MvPartitionStorage and TxStateStorage in PartitionAccess

tkalkirill commented on code in PR #1556: URL: https://github.com/apache/ignite-3/pull/1556#discussion_r1083919142 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/raft/snapshot/incoming/IncomingSnapshotCopier.java: ## @@ -226,18 +207,7 @@ private

[GitHub] [ignite-3] korlov42 commented on a diff in pull request #1524: IGNITE-18464 Sql. Colocated sort aggregates need to compose a plans with additional sort

korlov42 commented on code in PR #1524: URL: https://github.com/apache/ignite-3/pull/1524#discussion_r1083913680 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItSortAggregateTest.java: ## @@ -67,8 +70,134 @@ public void mapReduceAggregate() {

[GitHub] [ignite-3] korlov42 commented on a diff in pull request #1524: IGNITE-18464 Sql. Colocated sort aggregates need to compose a plans with additional sort

korlov42 commented on code in PR #1524: URL: https://github.com/apache/ignite-3/pull/1524#discussion_r1083913316 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItSortAggregateTest.java: ## @@ -48,12 +53,10 @@ static void initTestData() {

[GitHub] [ignite-3] korlov42 commented on a diff in pull request #1524: IGNITE-18464 Sql. Colocated sort aggregates need to compose a plans with additional sort

korlov42 commented on code in PR #1524: URL: https://github.com/apache/ignite-3/pull/1524#discussion_r1083908015 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/rel/SortAggregateNode.java: ## @@ -119,7 +124,11 @@ public void push(RowT row) throws

[GitHub] [ignite-3] korlov42 commented on a diff in pull request #1524: IGNITE-18464 Sql. Colocated sort aggregates need to compose a plans with additional sort

korlov42 commented on code in PR #1524: URL: https://github.com/apache/ignite-3/pull/1524#discussion_r1083907403 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/rel/SortAggregateNode.java: ## @@ -83,6 +84,10 @@ public SortAggregateNode(

[GitHub] [ignite-3] rpuch commented on a diff in pull request #1556: IGNITE-18593 Get rid of MvPartitionStorage and TxStateStorage in PartitionAccess

rpuch commented on code in PR #1556: URL: https://github.com/apache/ignite-3/pull/1556#discussion_r1083873554 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/raft/snapshot/PartitionAccess.java: ## @@ -32,27 +41,111 @@ public interface PartitionAccess

[GitHub] [ignite-3] tkalkirill opened a new pull request, #1562: IGNITE-18030 Integration of the new full rebalance API with IncomingSnapshotCopier

tkalkirill opened a new pull request, #1562: URL: https://github.com/apache/ignite-3/pull/1562 https://issues.apache.org/jira/browse/IGNITE-18030 -- 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

[GitHub] [ignite-3] AMashenkov merged pull request #1536: IGNITE-18207 Sql. Pushdown DISTINCT aggregate with no particular function

AMashenkov merged PR #1536: URL: https://github.com/apache/ignite-3/pull/1536 -- 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:

[GitHub] [ignite-3] korlov42 commented on a diff in pull request #1469: IGNITE-18227: refactoring scan nodes and add support RO index scans.

korlov42 commented on code in PR #1469: URL: https://github.com/apache/ignite-3/pull/1469#discussion_r1083836141 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/sql/api/ItSqlAsynchronousApiTest.java: ## @@ -343,37 +346,59 @@ public void

[GitHub] [ignite] Vladsz83 commented on a diff in pull request #10485: IGNITE-18232 : [IEP-80] Remove of CacheMetricsMXBean legacy JMX bean, v3 : keep JCache beans

Vladsz83 commented on code in PR #10485: URL: https://github.com/apache/ignite/pull/10485#discussion_r1083841496 ## modules/core/src/main/java/org/apache/ignite/IgniteCache.java: ## @@ -1617,20 +1616,6 @@ public IgniteFuture>> invokeAllAsync(Set

[GitHub] [ignite] Vladsz83 commented on a diff in pull request #10485: IGNITE-18232 : [IEP-80] Remove of CacheMetricsMXBean legacy JMX bean, v3 : keep JCache beans

Vladsz83 commented on code in PR #10485: URL: https://github.com/apache/ignite/pull/10485#discussion_r1083835143 ## modules/clients/src/test/java/org/apache/ignite/internal/processors/rest/RestMemcacheProtocolSelfTest.java: ## @@ -185,8 +185,8 @@ public void testReplace()

[GitHub] [ignite] nizhikov commented on a diff in pull request #10485: IGNITE-18232 : [IEP-80] Remove of CacheMetricsMXBean legacy JMX bean, v3 : keep JCache beans

nizhikov commented on code in PR #10485: URL: https://github.com/apache/ignite/pull/10485#discussion_r1083760009 ## modules/core/src/main/java/org/apache/ignite/IgniteCache.java: ## @@ -1617,20 +1616,6 @@ public IgniteFuture>> invokeAllAsync(Set

[GitHub] [ignite] nizhikov commented on a diff in pull request #10485: IGNITE-18232 : [IEP-80] Remove of CacheMetricsMXBean legacy JMX bean, v3 : keep JCache beans

nizhikov commented on code in PR #10485: URL: https://github.com/apache/ignite/pull/10485#discussion_r1083757319 ## modules/clients/src/test/java/org/apache/ignite/internal/processors/rest/RestMemcacheProtocolSelfTest.java: ## @@ -185,8 +185,8 @@ public void testReplace()

[GitHub] [ignite-3] ibessonov commented on a diff in pull request #1479: IGNITE-18360 Migrate storage to new Binary Tuple format

ibessonov commented on code in PR #1479: URL: https://github.com/apache/ignite-3/pull/1479#discussion_r1083750250 ## modules/schema/src/main/java/org/apache/ignite/internal/schema/TableRow.java: ## @@ -0,0 +1,105 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under