[GitHub] [ignite-3] SammyVimes commented on a diff in pull request #2266: IGNITE-19778 Restore components state on metastorage recovery

2023-07-05 Thread via GitHub
SammyVimes commented on code in PR #2266: URL: https://github.com/apache/ignite-3/pull/2266#discussion_r1252822420 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/configuration/ItDistributedConfigurationPropertiesTest.java: ## @@ -228,7 +228,7 @@ void start

[GitHub] [ignite-3] zstan commented on a diff in pull request #2279: IGNITE-19663 Add possibility to check transaction state

2023-07-05 Thread via GitHub
zstan commented on code in PR #2279: URL: https://github.com/apache/ignite-3/pull/2279#discussion_r1252833575 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/sql/api/ItCommonApiTest.java: ## @@ -232,7 +232,7 @@ public void testTxStateChangedOnErroneousOp()

[GitHub] [ignite] asdfgh19 opened a new issue, #10823: How to clean up invalid BinarySchema

2023-07-05 Thread via GitHub
asdfgh19 opened a new issue, #10823: URL: https://github.com/apache/ignite/issues/10823 I created a cache like this IgniteCache. In order to do upsert, I used EntryProcessor as follows. EntryProcessor ```java public class UpsertEntryProcessorV1 implements EntryProc

[GitHub] [ignite-3] sanpwc commented on a diff in pull request #2279: IGNITE-19663 Add possibility to check transaction state

2023-07-05 Thread via GitHub
sanpwc commented on code in PR #2279: URL: https://github.com/apache/ignite-3/pull/2279#discussion_r1252847240 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/sql/api/ItCommonApiTest.java: ## @@ -181,7 +181,7 @@ public void checkTimestampOperations() {

[GitHub] [ignite] ptupitsyn commented on issue #10823: How to clean up invalid BinarySchema

2023-07-05 Thread via GitHub
ptupitsyn commented on issue #10823: URL: https://github.com/apache/ignite/issues/10823#issuecomment-1621472014 > invalid BinarySchema It is not invalid. If a schema was used once, it has to be stored to deserialize that object later. You can try to minimize the number of uniqu

[GitHub] [ignite-3] korlov42 merged pull request #2280: IGNITE-17765 Sql. Introduce cache for parsed statements

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

[GitHub] [ignite] yurinaryshkin commented on a diff in pull request #10798: IGNITE-19675 [IEP-104] Implement WALIterator over ByteBuffer

2023-07-05 Thread via GitHub
yurinaryshkin commented on code in PR #10798: URL: https://github.com/apache/ignite/pull/10798#discussion_r1252934244 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/wal/io/SimpleFileInput.java: ## @@ -133,8 +133,8 @@ private void clearBuffe

[GitHub] [ignite] yurinaryshkin commented on a diff in pull request #10798: IGNITE-19675 [IEP-104] Implement WALIterator over ByteBuffer

2023-07-05 Thread via GitHub
yurinaryshkin commented on code in PR #10798: URL: https://github.com/apache/ignite/pull/10798#discussion_r1252941894 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/wal/ByteBufferWalIterator.java: ## @@ -0,0 +1,98 @@ +/* + * Licensed to the

[GitHub] [ignite-3] ptupitsyn opened a new pull request, #2290: IGNITE-19794 Fix tuple trimming in ClientMessagePacker

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

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10798: IGNITE-19675 [IEP-104] Implement WALIterator over ByteBuffer

2023-07-05 Thread via GitHub
sonarcloud[bot] commented on PR #10798: URL: https://github.com/apache/ignite/pull/10798#issuecomment-1621639854 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] denis-chudov commented on a diff in pull request #2279: IGNITE-19663 Add possibility to check transaction state

2023-07-05 Thread via GitHub
denis-chudov commented on code in PR #2279: URL: https://github.com/apache/ignite-3/pull/2279#discussion_r1252995910 ## modules/transactions/src/main/java/org/apache/ignite/internal/tx/TxManager.java: ## @@ -129,6 +128,14 @@ CompletableFuture cleanup( @TestOnly int fin

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10798: IGNITE-19675 [IEP-104] Implement WALIterator over ByteBuffer

2023-07-05 Thread via GitHub
sonarcloud[bot] commented on PR #10798: URL: https://github.com/apache/ignite/pull/10798#issuecomment-1621713138 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] timoninmaxim commented on a diff in pull request #10798: IGNITE-19675 [IEP-104] Implement WALIterator over ByteBuffer

2023-07-05 Thread via GitHub
timoninmaxim commented on code in PR #10798: URL: https://github.com/apache/ignite/pull/10798#discussion_r1253027588 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/wal/AbstractWalRecordsIteratorApdator.java: ## @@ -0,0 +1,90 @@ +/* + * Lice

[GitHub] [ignite] nizhikov opened a new pull request, #10824: IGNITE-19915 Remove obsolete IgniteCacheSnapshotManager

2023-07-05 Thread via GitHub
nizhikov opened a new pull request, #10824: URL: https://github.com/apache/ignite/pull/10824 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

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10824: IGNITE-19915 Remove obsolete IgniteCacheSnapshotManager

2023-07-05 Thread via GitHub
sonarcloud[bot] commented on PR #10824: URL: https://github.com/apache/ignite/pull/10824#issuecomment-1621818475 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 #10822: IGNITE-19912 Duplicated index creation using SQL and already existing QueryFields index leads to node start-up failure

2023-07-05 Thread via GitHub
sonarcloud[bot] commented on PR #10822: URL: https://github.com/apache/ignite/pull/10822#issuecomment-1621842488 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] J-Bakuli opened a new pull request, #10825: IGNITE-19163 Add logging of snapshot check

2023-07-05 Thread via GitHub
J-Bakuli opened a new pull request, #10825: URL: https://github.com/apache/ignite/pull/10825 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

[GitHub] [ignite] asdfgh19 commented on issue #10823: How to clean up invalid BinarySchema

2023-07-05 Thread via GitHub
asdfgh19 commented on issue #10823: URL: https://github.com/apache/ignite/issues/10823#issuecomment-1621907212 @ptupitsyn Thanks for your reply and suggestion! By analyzing BinaryObjectBuilderImpl#serializeTo, I found a little rule.  Writing fields out of order or updating some fields

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10824: IGNITE-19915 Remove obsolete IgniteCacheSnapshotManager

2023-07-05 Thread via GitHub
sonarcloud[bot] commented on PR #10824: URL: https://github.com/apache/ignite/pull/10824#issuecomment-1621915702 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 #10825: IGNITE-19163 Add logging of snapshot check

2023-07-05 Thread via GitHub
sonarcloud[bot] commented on PR #10825: URL: https://github.com/apache/ignite/pull/10825#issuecomment-1621919913 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-3] sanpwc merged pull request #2279: IGNITE-19663 Add possibility to check transaction state

2023-07-05 Thread via GitHub
sanpwc merged PR #2279: URL: https://github.com/apache/ignite-3/pull/2279 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@igni

[GitHub] [ignite-3] sk0x50 merged pull request #2287: IGNITE-19864 Introduce TraceableException

2023-07-05 Thread via GitHub
sk0x50 merged PR #2287: URL: https://github.com/apache/ignite-3/pull/2287 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@igni

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10824: IGNITE-19915 Remove obsolete IgniteCacheSnapshotManager

2023-07-05 Thread via GitHub
sonarcloud[bot] commented on PR #10824: URL: https://github.com/apache/ignite/pull/10824#issuecomment-1621985743 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] sk0x50 opened a new pull request, #2291: IGNITE-19865 Fix propagating trace identifier

2023-07-05 Thread via GitHub
sk0x50 opened a new pull request, #2291: URL: https://github.com/apache/ignite-3/pull/2291 (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-ma

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10824: IGNITE-19915 Remove obsolete IgniteCacheSnapshotManager

2023-07-05 Thread via GitHub
sonarcloud[bot] commented on PR #10824: URL: https://github.com/apache/ignite/pull/10824#issuecomment-1622006935 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] ptupitsyn commented on issue #10823: How to clean up invalid BinarySchema

2023-07-05 Thread via GitHub
ptupitsyn commented on issue #10823: URL: https://github.com/apache/ignite/issues/10823#issuecomment-1622097935 Thanks for posting the solution. Yes, there is a trade-off - create more schemas, or waste some space for nulls. Can you also describe your use case a little bit please? 100

[GitHub] [ignite-3] denis-chudov opened a new pull request, #2292: IGNITE-18959

2023-07-05 Thread via GitHub
denis-chudov opened a new pull request, #2292: URL: https://github.com/apache/ignite-3/pull/2292 (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

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10824: IGNITE-19915 Remove obsolete IgniteCacheSnapshotManager

2023-07-05 Thread via GitHub
sonarcloud[bot] commented on PR #10824: URL: https://github.com/apache/ignite/pull/10824#issuecomment-1622388115 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 #10824: IGNITE-19915 Remove obsolete IgniteCacheSnapshotManager

2023-07-05 Thread via GitHub
sonarcloud[bot] commented on PR #10824: URL: https://github.com/apache/ignite/pull/10824#issuecomment-1622445177 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] ptupitsyn merged pull request #2289: IGNITE-19756 Java client: fix colocation column order

2023-07-05 Thread via GitHub
ptupitsyn merged PR #2289: URL: https://github.com/apache/ignite-3/pull/2289 -- 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-3] ptupitsyn merged pull request #2290: IGNITE-19794 Remove BinaryTuple slicing in ClientMessagePacker

2023-07-05 Thread via GitHub
ptupitsyn merged PR #2290: URL: https://github.com/apache/ignite-3/pull/2290 -- 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-3] zstan merged pull request #2275: IGNITE-19644 Change DROP|ADD COLUMN IF (NOT) EXISTS syntax

2023-07-05 Thread via GitHub
zstan merged PR #2275: URL: https://github.com/apache/ignite-3/pull/2275 -- 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] ibessonov commented on a diff in pull request #2266: IGNITE-19778 Restore components state on metastorage recovery

2023-07-06 Thread via GitHub
ibessonov commented on code in PR #2266: URL: https://github.com/apache/ignite-3/pull/2266#discussion_r1254029147 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/TableManager.java: ## @@ -1234,7 +1236,7 @@ private CompletableFuture createTableLocally(

[GitHub] [ignite-3] zstan merged pull request #2241: IGNITE-19730 Sql. ExecutionServiceImplTest different tests failed

2023-07-06 Thread via GitHub
zstan merged PR #2241: URL: https://github.com/apache/ignite-3/pull/2241 -- 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] sanpwc merged pull request #2288: IGNITE-19581 add restoring of zones state after restart taking into account zone's filter updates

2023-07-06 Thread via GitHub
sanpwc merged PR #2288: URL: https://github.com/apache/ignite-3/pull/2288 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@igni

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
AMashenkov commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254110934 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/api/SessionImpl.java: ## @@ -330,7 +329,13 @@ public Publisher executeBatchReactive(@Nullable Tr

[GitHub] [ignite-extensions] nva opened a new pull request, #222: IGNITE-19921 Add thin client support for Spring Session.

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

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
AMashenkov commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254127404 ## modules/core/src/main/java/org/apache/ignite/lang/ErrorGroups.java: ## @@ -117,92 +117,35 @@ public static class Sql { /** Session not found error. */

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
AMashenkov commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254129402 ## modules/core/src/main/java/org/apache/ignite/lang/ErrorGroups.java: ## @@ -117,92 +117,35 @@ public static class Sql { /** Session not found error. */

[GitHub] [ignite-3] ptupitsyn opened a new pull request, #2293: IGNITE-19667 Make ClientTableCommon.readTable async

2023-07-06 Thread via GitHub
ptupitsyn opened a new pull request, #2293: URL: https://github.com/apache/ignite-3/pull/2293 * Make `ClientTableCommon.readTable` async to avoid blocking Netty threads * Change `IgniteTablesInternal#assignments` to async too -- This is an automated message from the Apache Git Service.

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
AMashenkov commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254129402 ## modules/core/src/main/java/org/apache/ignite/lang/ErrorGroups.java: ## @@ -117,92 +117,35 @@ public static class Sql { /** Session not found error. */

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
AMashenkov commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254135094 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/ExecutionServiceImpl.java: ## @@ -311,11 +310,11 @@ private static RuntimeException

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
AMashenkov commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254148000 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/rel/AbstractNode.java: ## @@ -161,7 +161,7 @@ protected void checkState() throws Exce

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
AMashenkov commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254148000 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/rel/AbstractNode.java: ## @@ -161,7 +161,7 @@ protected void checkState() throws Exce

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
AMashenkov commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254151215 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/rel/HashAggregateNode.java: ## @@ -77,11 +75,9 @@ public HashAggregateNode(

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
AMashenkov commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254155001 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/rel/Inbox.java: ## @@ -365,7 +365,9 @@ private void onNodeLeft0(String nodeName) thro

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
AMashenkov commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254155001 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/rel/Inbox.java: ## @@ -365,7 +365,9 @@ private void onNodeLeft0(String nodeName) thro

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
AMashenkov commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254161133 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/prepare/IgnitePlanner.java: ## @@ -264,7 +264,7 @@ public RelRoot expandView(

[GitHub] [ignite-3] lowka commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
lowka commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254162463 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/rel/HashAggregateNode.java: ## @@ -77,11 +75,9 @@ public HashAggregateNode( this.

[GitHub] [ignite-3] lowka commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
lowka commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254169795 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/rel/HashAggregateNode.java: ## @@ -77,11 +75,9 @@ public HashAggregateNode( this.

[GitHub] [ignite-3] lowka commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
lowka commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254169795 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/rel/HashAggregateNode.java: ## @@ -77,11 +75,9 @@ public HashAggregateNode( this.

[GitHub] [ignite-3] ibessonov commented on a diff in pull request #2266: IGNITE-19778 Restore components state on metastorage recovery

2023-07-06 Thread via GitHub
ibessonov commented on code in PR #2266: URL: https://github.com/apache/ignite-3/pull/2266#discussion_r1254197799 ## modules/metastorage/src/main/java/org/apache/ignite/internal/metastorage/impl/MetaStorageManagerImpl.java: ## @@ -824,19 +830,7 @@ private CompletableFuture onRe

[GitHub] [ignite-3] lowka commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
lowka commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254221227 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/rel/Inbox.java: ## @@ -365,7 +365,9 @@ private void onNodeLeft0(String nodeName) throws Ex

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2282: IGNITE-17298 Sql. Support BOOLEAN type.

2023-07-06 Thread via GitHub
AMashenkov commented on code in PR #2282: URL: https://github.com/apache/ignite-3/pull/2282#discussion_r1254228854 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItSecondaryIndexTest.java: ## @@ -993,4 +993,60 @@ public void testSaturatedBoundsH

[GitHub] [ignite-3] lowka commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
lowka commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254233871 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/prepare/IgnitePlanner.java: ## @@ -264,7 +264,7 @@ public RelRoot expandView( sqlN

[GitHub] [ignite-3] lowka commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
lowka commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254241840 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/api/SessionImpl.java: ## @@ -330,7 +329,13 @@ public Publisher executeBatchReactive(@Nullable Transac

[GitHub] [ignite-3] lowka commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
lowka commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254250115 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/ExecutionServiceImpl.java: ## @@ -311,11 +310,11 @@ private static RuntimeException conve

[GitHub] [ignite-3] lowka commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
lowka commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254250115 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/ExecutionServiceImpl.java: ## @@ -311,11 +310,11 @@ private static RuntimeException conve

[GitHub] [ignite-3] lowka commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
lowka commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254251382 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/rel/AbstractNode.java: ## @@ -161,7 +161,7 @@ protected void checkState() throws Exception

[GitHub] [ignite-3] lowka commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
lowka commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254253381 ## modules/core/src/main/java/org/apache/ignite/lang/ErrorGroups.java: ## @@ -117,92 +117,35 @@ public static class Sql { /** Session not found error. */

[GitHub] [ignite-3] lowka commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
lowka commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254257991 ## modules/core/src/main/java/org/apache/ignite/lang/ErrorGroups.java: ## @@ -117,92 +117,35 @@ public static class Sql { /** Session not found error. */

[GitHub] [ignite-3] lowka commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
lowka commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254260378 ## modules/core/src/main/java/org/apache/ignite/lang/ErrorGroups.java: ## @@ -117,92 +117,35 @@ public static class Sql { /** Session not found error. */

[GitHub] [ignite-3] lowka opened a new pull request, #2294: [IGNITE-19785] Sql. Rework execution of 2-phase aggregates.

2023-07-06 Thread via GitHub
lowka opened a new pull request, #2294: URL: https://github.com/apache/ignite-3/pull/2294 (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] ptupitsyn merged pull request #2293: IGNITE-19667 Make ClientTableCommon.readTable async

2023-07-06 Thread via GitHub
ptupitsyn merged PR #2293: URL: https://github.com/apache/ignite-3/pull/2293 -- 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-3] zstan commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
zstan commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254286874 ## modules/client-handler/src/test/java/org/apache/ignite/client/handler/JdbcQueryEventHandlerImplTest.java: ## @@ -299,4 +299,13 @@ private long acquireConnectionId() {

[GitHub] [ignite-3] sashapolo opened a new pull request, #2295: IGNITE-19810 Remove excessive BinaryRow buffer copying

2023-07-06 Thread via GitHub
sashapolo opened a new pull request, #2295: URL: https://github.com/apache/ignite-3/pull/2295 https://issues.apache.org/jira/browse/IGNITE-19810 -- 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] nizhikov closed pull request #10451: IGNITE-18312 Control script migration to thin client

2023-07-06 Thread via GitHub
nizhikov closed pull request #10451: IGNITE-18312 Control script migration to thin client URL: https://github.com/apache/ignite/pull/10451 -- 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 specifi

[GitHub] [ignite] nizhikov closed pull request #10475: IGNITE-18520 Support stopWarmup command in thin client

2023-07-06 Thread via GitHub
nizhikov closed pull request #10475: IGNITE-18520 Support stopWarmup command in thin client URL: https://github.com/apache/ignite/pull/10475 -- 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 speci

[GitHub] [ignite-3] vldpyatkov merged pull request #2141: IGNITE-19578 Decrease count of lease messages to meta storage

2023-07-06 Thread via GitHub
vldpyatkov merged PR #2141: URL: https://github.com/apache/ignite-3/pull/2141 -- 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] alievmirza commented on a diff in pull request #2266: IGNITE-19778 Restore components state on metastorage recovery

2023-07-06 Thread via GitHub
alievmirza commented on code in PR #2266: URL: https://github.com/apache/ignite-3/pull/2266#discussion_r1254364235 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/distribution/zones/ItDistributionZonesFilterTest.java: ## @@ -95,6 +96,7 @@ protected int init

[GitHub] [ignite-3] alievmirza commented on a diff in pull request #2266: IGNITE-19778 Restore components state on metastorage recovery

2023-07-06 Thread via GitHub
alievmirza commented on code in PR #2266: URL: https://github.com/apache/ignite-3/pull/2266#discussion_r1254364235 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/distribution/zones/ItDistributionZonesFilterTest.java: ## @@ -95,6 +96,7 @@ protected int init

[GitHub] [ignite-3] lowka commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
lowka commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254416915 ## modules/client-handler/src/test/java/org/apache/ignite/client/handler/JdbcQueryEventHandlerImplTest.java: ## @@ -299,4 +299,13 @@ private long acquireConnectionId() {

[GitHub] [ignite-website] timoninmaxim opened a new pull request, #169: Update ignite-2.14 release notes

2023-07-06 Thread via GitHub
timoninmaxim opened a new pull request, #169: URL: https://github.com/apache/ignite-website/pull/169 (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 unsubsc

[GitHub] [ignite-3] SammyVimes commented on a diff in pull request #2266: IGNITE-19778 Restore components state on metastorage recovery

2023-07-06 Thread via GitHub
SammyVimes commented on code in PR #2266: URL: https://github.com/apache/ignite-3/pull/2266#discussion_r1254451413 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/distribution/zones/ItDistributionZonesFilterTest.java: ## @@ -95,6 +96,7 @@ protected int init

[GitHub] [ignite-website] ptupitsyn commented on a diff in pull request #169: Update ignite-2.14 release notes

2023-07-06 Thread via GitHub
ptupitsyn commented on code in PR #169: URL: https://github.com/apache/ignite-website/pull/169#discussion_r1254460274 ## releases/2.14.0/release_notes.html: ## @@ -47,6 +47,11 @@ Apache Ignite 2.14.0 WARNINGS + +Java thin clients retrieves Binary

[GitHub] [ignite-3] lowka commented on a diff in pull request #2282: IGNITE-17298 Sql. Support BOOLEAN type.

2023-07-06 Thread via GitHub
lowka commented on code in PR #2282: URL: https://github.com/apache/ignite-3/pull/2282#discussion_r1254483739 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItSecondaryIndexTest.java: ## @@ -993,4 +993,60 @@ public void testSaturatedBoundsHashIn

[GitHub] [ignite] timoninmaxim opened a new pull request, #10826: IGNITE-17651 Update Ignite 2.14 release notes

2023-07-06 Thread via GitHub
timoninmaxim opened a new pull request, #10826: URL: https://github.com/apache/ignite/pull/10826 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] alievmirza commented on a diff in pull request #2266: IGNITE-19778 Restore components state on metastorage recovery

2023-07-06 Thread via GitHub
alievmirza commented on code in PR #2266: URL: https://github.com/apache/ignite-3/pull/2266#discussion_r1254514488 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/distribution/zones/ItDistributionZonesFilterTest.java: ## @@ -95,6 +96,7 @@ protected int init

[GitHub] [ignite-website] timoninmaxim merged pull request #169: Update ignite-2.14 release notes

2023-07-06 Thread via GitHub
timoninmaxim merged PR #169: URL: https://github.com/apache/ignite-website/pull/169 -- 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-unsubs

[GitHub] [ignite-3] AMashenkov opened a new pull request, #2296: IGNITE-19924 Test shouldn't guess id of newly created catalog object

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

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10826: IGNITE-17651 Update Ignite 2.14 release notes

2023-07-06 Thread via GitHub
sonarcloud[bot] commented on PR #10826: URL: https://github.com/apache/ignite/pull/10826#issuecomment-1623779435 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 merged pull request #10826: IGNITE-17651 Update Ignite 2.14 release notes

2023-07-06 Thread via GitHub
timoninmaxim merged PR #10826: URL: https://github.com/apache/ignite/pull/10826 -- 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] yurinaryshkin commented on a diff in pull request #10798: IGNITE-19675 [IEP-104] Implement WALIterator over ByteBuffer

2023-07-06 Thread via GitHub
yurinaryshkin commented on code in PR #10798: URL: https://github.com/apache/ignite/pull/10798#discussion_r1254558770 ## modules/core/src/test/java/org/apache/ignite/internal/processors/cache/persistence/wal/ByteBufferWalIteratorTest.java: ## @@ -0,0 +1,251 @@ +/* + * Licensed t

[GitHub] [ignite] yurinaryshkin commented on a diff in pull request #10798: IGNITE-19675 [IEP-104] Implement WALIterator over ByteBuffer

2023-07-06 Thread via GitHub
yurinaryshkin commented on code in PR #10798: URL: https://github.com/apache/ignite/pull/10798#discussion_r1254562746 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/wal/ByteBufferWalIterator.java: ## @@ -0,0 +1,98 @@ +/* + * Licensed to the

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10798: IGNITE-19675 [IEP-104] Implement WALIterator over ByteBuffer

2023-07-06 Thread via GitHub
sonarcloud[bot] commented on PR #10798: URL: https://github.com/apache/ignite/pull/10798#issuecomment-1623861415 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] timoninmaxim commented on a diff in pull request #10798: IGNITE-19675 [IEP-104] Implement WALIterator over ByteBuffer

2023-07-06 Thread via GitHub
timoninmaxim commented on code in PR #10798: URL: https://github.com/apache/ignite/pull/10798#discussion_r1254574971 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/wal/ByteBufferBackedDataInputImpl.java: ## @@ -44,22 +44,22 @@ public ByteBu

[GitHub] [ignite-3] xtern commented on a diff in pull request #2282: IGNITE-17298 Sql. Support BOOLEAN type.

2023-07-06 Thread via GitHub
xtern commented on code in PR #2282: URL: https://github.com/apache/ignite-3/pull/2282#discussion_r1254590091 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItSecondaryIndexTest.java: ## @@ -993,4 +993,60 @@ public void testSaturatedBoundsHashIn

[GitHub] [ignite-3] xtern commented on a diff in pull request #2282: IGNITE-17298 Sql. Support BOOLEAN type.

2023-07-06 Thread via GitHub
xtern commented on code in PR #2282: URL: https://github.com/apache/ignite-3/pull/2282#discussion_r1254590091 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItSecondaryIndexTest.java: ## @@ -993,4 +993,60 @@ public void testSaturatedBoundsHashIn

[GitHub] [ignite-3] AMashenkov opened a new pull request, #2297: IGNITE-19927 Improve test coverage for CREATE TABLE

2023-07-06 Thread via GitHub
AMashenkov opened a new pull request, #2297: URL: https://github.com/apache/ignite-3/pull/2297 https://issues.apache.org/jira/browse/IGNITE-19927 -- 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-3] AMashenkov commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
AMashenkov commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254614192 ## modules/core/src/main/java/org/apache/ignite/lang/ErrorGroups.java: ## @@ -117,92 +117,35 @@ public static class Sql { /** Session not found error. */

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
AMashenkov commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254620390 ## modules/core/src/main/java/org/apache/ignite/lang/ErrorGroups.java: ## @@ -117,92 +117,35 @@ public static class Sql { /** Session not found error. */

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10778: IGNITE-19622 Add realtime CDC buffer

2023-07-06 Thread via GitHub
sonarcloud[bot] commented on PR #10778: URL: https://github.com/apache/ignite/pull/10778#issuecomment-1623902717 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 a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
AMashenkov commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254627643 ## modules/core/src/main/java/org/apache/ignite/lang/ErrorGroups.java: ## @@ -117,92 +117,35 @@ public static class Sql { /** Session not found error. */

[GitHub] [ignite] yurinaryshkin commented on a diff in pull request #10798: IGNITE-19675 [IEP-104] Implement WALIterator over ByteBuffer

2023-07-06 Thread via GitHub
yurinaryshkin commented on code in PR #10798: URL: https://github.com/apache/ignite/pull/10798#discussion_r1254627697 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/wal/ByteBufferBackedDataInputImpl.java: ## @@ -44,22 +44,22 @@ public ByteB

[GitHub] [ignite] yurinaryshkin commented on a diff in pull request #10798: IGNITE-19675 [IEP-104] Implement WALIterator over ByteBuffer

2023-07-06 Thread via GitHub
yurinaryshkin commented on code in PR #10798: URL: https://github.com/apache/ignite/pull/10798#discussion_r1254634575 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/wal/serializer/RecordV2Serializer.java: ## @@ -266,7 +265,7 @@ private stat

[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2271: IGNITE-19862: Revise error codes for SQL module

2023-07-06 Thread via GitHub
AMashenkov commented on code in PR #2271: URL: https://github.com/apache/ignite-3/pull/2271#discussion_r1254640627 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/ExecutionServiceImpl.java: ## @@ -311,11 +310,11 @@ private static RuntimeException

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10798: IGNITE-19675 [IEP-104] Implement WALIterator over ByteBuffer

2023-07-06 Thread via GitHub
sonarcloud[bot] commented on PR #10798: URL: https://github.com/apache/ignite/pull/10798#issuecomment-1623953771 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] alex-plekhanov opened a new pull request, #10827: IGNITE-19748 SQL Calcite: Queries timeout support

2023-07-06 Thread via GitHub
alex-plekhanov opened a new pull request, #10827: URL: https://github.com/apache/ignite/pull/10827 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] asfgit closed pull request #10816: IGNITE-19811 Fix continuous queries backup ack message sending for ex…

2023-07-06 Thread via GitHub
asfgit closed pull request #10816: IGNITE-19811 Fix continuous queries backup ack message sending for ex… URL: https://github.com/apache/ignite/pull/10816 -- 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 g

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10827: IGNITE-19748 SQL Calcite: Queries timeout support

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

<    7   8   9   10   11   12   13   14   15   16   >