[GitHub] [ignite] sonarcloud[bot] commented on pull request #10775: IGNITE-19676 Command JMX Invoker

2023-06-26 Thread via GitHub
sonarcloud[bot] commented on PR #10775: URL: https://github.com/apache/ignite/pull/10775#issuecomment-1607847782 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] alievmirza commented on a diff in pull request #2201: IGNITE-19736 Do not cancel tasks in DistributionZoneManager#executor if they were created by immediate scaleUp/scaleDown event

2023-06-26 Thread via GitHub
alievmirza commented on code in PR #2201: URL: https://github.com/apache/ignite-3/pull/2201#discussion_r1242432274 ## modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DistributionZoneManager.java: ## @@ -282,8 +282,10 @@ public DistributionZo

[GitHub] [ignite-3] sergeyuttsel commented on a diff in pull request #2201: IGNITE-19736 Do not cancel tasks in DistributionZoneManager#executor if they were created by immediate scaleUp/scaleDown eve

2023-06-26 Thread via GitHub
sergeyuttsel commented on code in PR #2201: URL: https://github.com/apache/ignite-3/pull/2201#discussion_r1242545958 ## modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DistributionZoneManager.java: ## @@ -282,8 +282,10 @@ public Distribution

[GitHub] [ignite-3] sashapolo commented on a diff in pull request #2237: IGNITE-19777 Perform local metastorage recovery

2023-06-26 Thread via GitHub
sashapolo commented on code in PR #2237: URL: https://github.com/apache/ignite-3/pull/2237#discussion_r1242239452 ## modules/metastorage/src/main/java/org/apache/ignite/internal/metastorage/impl/MetaStorageManagerImpl.java: ## @@ -148,6 +155,77 @@ public MetaStorageManagerImpl(

[GitHub] [ignite-3] sashapolo commented on a diff in pull request #2237: IGNITE-19777 Perform local metastorage recovery

2023-06-26 Thread via GitHub
sashapolo commented on code in PR #2237: URL: https://github.com/apache/ignite-3/pull/2237#discussion_r1242571976 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/runner/app/ItIgniteNodeRestartTest.java: ## @@ -719,11 +732,66 @@ public void nodeWithDataTest(

[GitHub] [ignite-3] sashapolo commented on a diff in pull request #2239: IGNITE-19199 Propagate safe time when Meta Storage is idle

2023-06-26 Thread via GitHub
sashapolo commented on code in PR #2239: URL: https://github.com/apache/ignite-3/pull/2239#discussion_r1242573594 ## modules/metastorage/src/main/java/org/apache/ignite/internal/metastorage/impl/MetaStorageLeaderElectionListener.java: ## @@ -62,87 +66,108 @@ public class MetaSto

[GitHub] [ignite] anton-vinogradov opened a new pull request, #10806: IGNITE-19846 IgniteTxAdapter initial cleanup

2023-06-26 Thread via GitHub
anton-vinogradov opened a new pull request, #10806: URL: https://github.com/apache/ignite/pull/10806 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] sonarcloud[bot] commented on pull request #10806: IGNITE-19846 IgniteTxAdapter initial cleanup

2023-06-26 Thread via GitHub
sonarcloud[bot] commented on PR #10806: URL: https://github.com/apache/ignite/pull/10806#issuecomment-160782 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] anton-vinogradov opened a new pull request, #10807: IGNITE-19847 IgniteTxAdapter.TxShadow removal

2023-06-26 Thread via GitHub
anton-vinogradov opened a new pull request, #10807: URL: https://github.com/apache/ignite/pull/10807 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] sonarcloud[bot] commented on pull request #10807: IGNITE-19847 IgniteTxAdapter.TxShadow removal

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

2023-06-26 Thread via GitHub
sonarcloud[bot] commented on PR #10798: URL: https://github.com/apache/ignite/pull/10798#issuecomment-1608040247 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] isapego opened a new pull request, #2257: IGNITE-19625 C++ Compute API supports deployment units

2023-06-26 Thread via GitHub
isapego opened a new pull request, #2257: URL: https://github.com/apache/ignite-3/pull/2257 - Changed Compute API in C++ client; - Fixed existing tests; - Added new tests. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [ignite-3] sanpwc commented on a diff in pull request #2225: IGNITE-19539 Added exception mapper utility

2023-06-26 Thread via GitHub
sanpwc commented on code in PR #2225: URL: https://github.com/apache/ignite-3/pull/2225#discussion_r1243209920 ## modules/api/src/main/java/org/apache/ignite/lang/IgniteExceptionMapperUtil.java: ## @@ -0,0 +1,170 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

[GitHub] [ignite-3] isapego merged pull request #2257: IGNITE-19625 C++ Compute API supports deployment units

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

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #2237: IGNITE-19777 Perform local metastorage recovery

2023-06-26 Thread via GitHub
tkalkirill commented on code in PR #2237: URL: https://github.com/apache/ignite-3/pull/2237#discussion_r1243214953 ## modules/metastorage/src/main/java/org/apache/ignite/internal/metastorage/server/persistence/RocksDbKeyValueStorage.java: ## @@ -170,6 +172,12 @@ public class Roc

[GitHub] [ignite-3] sk0x50 commented on a diff in pull request #2225: IGNITE-19539 Added exception mapper utility

2023-06-27 Thread via GitHub
sk0x50 commented on code in PR #2225: URL: https://github.com/apache/ignite-3/pull/2225#discussion_r1243236656 ## modules/api/src/main/java/org/apache/ignite/lang/IgniteExceptionMapperUtil.java: ## @@ -0,0 +1,170 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

[GitHub] [ignite-3] xtern commented on a diff in pull request #2226: IGNITE-17765 Sql. Introduce cache for parsed statements

2023-06-27 Thread via GitHub
xtern commented on code in PR #2226: URL: https://github.com/apache/ignite-3/pull/2226#discussion_r1241091906 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/prepare/PrepareServiceImpl.java: ## @@ -141,8 +164,59 @@ public void stop() throws Exception {

[GitHub] [ignite-3] SammyVimes commented on a diff in pull request #2237: IGNITE-19777 Perform local metastorage recovery

2023-06-27 Thread via GitHub
SammyVimes commented on code in PR #2237: URL: https://github.com/apache/ignite-3/pull/2237#discussion_r1243268301 ## modules/metastorage/src/main/java/org/apache/ignite/internal/metastorage/server/persistence/RocksDbKeyValueStorage.java: ## @@ -311,6 +318,12 @@ protected void c

[GitHub] [ignite-3] SammyVimes commented on a diff in pull request #2237: IGNITE-19777 Perform local metastorage recovery

2023-06-27 Thread via GitHub
SammyVimes commented on code in PR #2237: URL: https://github.com/apache/ignite-3/pull/2237#discussion_r1243273689 ## modules/metastorage/src/main/java/org/apache/ignite/internal/metastorage/server/persistence/RocksDbKeyValueStorage.java: ## @@ -170,6 +172,12 @@ public class Roc

[GitHub] [ignite-3] vldpyatkov opened a new pull request, #2258: IGNITE-19393 Java thin 3.0: testAccessLockedKeyTimesOut is flaky

2023-06-27 Thread via GitHub
vldpyatkov opened a new pull request, #2258: URL: https://github.com/apache/ignite-3/pull/2258 (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] sanpwc commented on a diff in pull request #2225: IGNITE-19539 Added exception mapper utility

2023-06-27 Thread via GitHub
sanpwc commented on code in PR #2225: URL: https://github.com/apache/ignite-3/pull/2225#discussion_r1243283024 ## modules/api/src/main/java/org/apache/ignite/lang/IgniteExceptionMapperUtil.java: ## @@ -0,0 +1,170 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

[GitHub] [ignite-3] alievmirza opened a new pull request, #2259: IGNITE-19688 use waitAndGetLeader instead of waitLeader and getLeader

2023-06-27 Thread via GitHub
alievmirza opened a new pull request, #2259: URL: https://github.com/apache/ignite-3/pull/2259 https://issues.apache.org/jira/browse/IGNITE-19688 -- 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] xtern commented on a diff in pull request #2226: IGNITE-17765 Sql. Introduce cache for parsed statements

2023-06-27 Thread via GitHub
xtern commented on code in PR #2226: URL: https://github.com/apache/ignite-3/pull/2226#discussion_r1243296683 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/prepare/PrepareService.java: ## @@ -29,5 +29,5 @@ public interface PrepareService extends Lifec

[GitHub] [ignite-3] xtern commented on a diff in pull request #2226: IGNITE-17765 Sql. Introduce cache for parsed statements

2023-06-27 Thread via GitHub
xtern commented on code in PR #2226: URL: https://github.com/apache/ignite-3/pull/2226#discussion_r1243298133 ## modules/sql-engine/src/test/java/org/apache/ignite/internal/sql/engine/planner/PrepareServiceSelfTest.java: ## @@ -0,0 +1,273 @@ +/* + * Licensed to the Apache Softwa

[GitHub] [ignite-3] tkalkirill opened a new pull request, #2260: IGNITE-19801 Configuration prematurely executes metastore revision update listener

2023-06-27 Thread via GitHub
tkalkirill opened a new pull request, #2260: URL: https://github.com/apache/ignite-3/pull/2260 https://issues.apache.org/jira/browse/IGNITE-19801 -- 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] ibessonov commented on a diff in pull request #2239: IGNITE-19199 Propagate safe time when Meta Storage is idle

2023-06-27 Thread via GitHub
ibessonov commented on code in PR #2239: URL: https://github.com/apache/ignite-3/pull/2239#discussion_r1243253962 ## modules/metastorage/src/main/java/org/apache/ignite/internal/metastorage/server/time/ClusterTimeImpl.java: ## @@ -119,48 +146,81 @@ public void updateSafeTime(Hyb

[GitHub] [ignite-3] sk0x50 merged pull request #2201: IGNITE-19736 Do not cancel tasks in DistributionZoneManager#executor if they were created by immediate scaleUp/scaleDown events. Avoid concurrent

2023-06-27 Thread via GitHub
sk0x50 merged PR #2201: URL: https://github.com/apache/ignite-3/pull/2201 -- 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] tkalkirill opened a new pull request, #2261: IGNITE-19851 Add MetaStorage update update listeners and use instead of configuration revision update listeners

2023-06-27 Thread via GitHub
tkalkirill opened a new pull request, #2261: URL: https://github.com/apache/ignite-3/pull/2261 https://issues.apache.org/jira/browse/IGNITE-19851 -- 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] ibessonov commented on a diff in pull request #2261: IGNITE-19851 Add MetaStorage update update listeners and use instead of configuration revision update listeners

2023-06-27 Thread via GitHub
ibessonov commented on code in PR #2261: URL: https://github.com/apache/ignite-3/pull/2261#discussion_r1243365841 ## modules/metastorage-api/src/main/java/org/apache/ignite/internal/metastorage/MetaStorageManager.java: ## @@ -216,4 +216,10 @@ public interface MetaStorageManager

[GitHub] [ignite-3] PakhomovAlexander commented on a diff in pull request #2238: IGNITE-19694 Introduce deployment unit operation ID

2023-06-27 Thread via GitHub
PakhomovAlexander commented on code in PR #2238: URL: https://github.com/apache/ignite-3/pull/2238#discussion_r124338 ## modules/code-deployment/src/main/java/org/apache/ignite/internal/deployunit/metastore/DeploymentUnitFailover.java: ## @@ -113,12 +116,43 @@ private void p

[GitHub] [ignite-3] PakhomovAlexander merged pull request #2250: IGNITE-19689 fix ItClusterManagerTest.testClusterConfigurationIsRemovedFromClusterStateAfterUpdating

2023-06-27 Thread via GitHub
PakhomovAlexander merged PR #2250: URL: https://github.com/apache/ignite-3/pull/2250 -- 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 #10762: IGNITE-19158 Improve message about received partition file during sna…

2023-06-27 Thread via GitHub
sonarcloud[bot] commented on PR #10762: URL: https://github.com/apache/ignite/pull/10762#issuecomment-1609121571 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] Pochatkin commented on a diff in pull request #2238: IGNITE-19694 Introduce deployment unit operation ID

2023-06-27 Thread via GitHub
Pochatkin commented on code in PR #2238: URL: https://github.com/apache/ignite-3/pull/2238#discussion_r1243469230 ## modules/code-deployment/src/main/java/org/apache/ignite/internal/deployunit/metastore/DeploymentUnitFailover.java: ## @@ -113,12 +116,43 @@ private void processSt

[GitHub] [ignite-3] xtern commented on a diff in pull request #2226: IGNITE-17765 Sql. Introduce cache for parsed statements

2023-06-27 Thread via GitHub
xtern commented on code in PR #2226: URL: https://github.com/apache/ignite-3/pull/2226#discussion_r1243470496 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/prepare/PrepareServiceImpl.java: ## @@ -296,4 +364,67 @@ private ResultSetMetadata resultSetMet

[GitHub] [ignite-3] xtern commented on a diff in pull request #2226: IGNITE-17765 Sql. Introduce cache for parsed statements

2023-06-27 Thread via GitHub
xtern commented on code in PR #2226: URL: https://github.com/apache/ignite-3/pull/2226#discussion_r1243470496 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/prepare/PrepareServiceImpl.java: ## @@ -296,4 +364,67 @@ private ResultSetMetadata resultSetMet

[GitHub] [ignite-3] xtern commented on a diff in pull request #2226: IGNITE-17765 Sql. Introduce cache for parsed statements

2023-06-27 Thread via GitHub
xtern commented on code in PR #2226: URL: https://github.com/apache/ignite-3/pull/2226#discussion_r1243470496 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/prepare/PrepareServiceImpl.java: ## @@ -296,4 +364,67 @@ private ResultSetMetadata resultSetMet

[GitHub] [ignite-3] xtern commented on a diff in pull request #2226: IGNITE-17765 Sql. Introduce cache for parsed statements

2023-06-27 Thread via GitHub
xtern commented on code in PR #2226: URL: https://github.com/apache/ignite-3/pull/2226#discussion_r1243470496 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/prepare/PrepareServiceImpl.java: ## @@ -296,4 +364,67 @@ private ResultSetMetadata resultSetMet

[GitHub] [ignite-3] SammyVimes merged pull request #2237: IGNITE-19777 Perform local metastorage recovery

2023-06-27 Thread via GitHub
SammyVimes merged PR #2237: URL: https://github.com/apache/ignite-3/pull/2237 -- 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] sonarcloud[bot] commented on pull request #10798: IGNITE-19675 [IEP-104] Implement WALIterator over ByteBuffer

2023-06-27 Thread via GitHub
sonarcloud[bot] commented on PR #10798: URL: https://github.com/apache/ignite/pull/10798#issuecomment-1609204690 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] nizhikov merged pull request #10775: IGNITE-19676 Command JMX Invoker

2023-06-27 Thread via GitHub
nizhikov merged PR #10775: URL: https://github.com/apache/ignite/pull/10775 -- 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-3] ptupitsyn opened a new pull request, #2262: IGNITE-19610 .NET: Add data streamer metrics

2023-06-27 Thread via GitHub
ptupitsyn opened a new pull request, #2262: URL: https://github.com/apache/ignite-3/pull/2262 https://issues.apache.org/jira/browse/IGNITE-19610 -- 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] sonarcloud[bot] commented on pull request #10775: IGNITE-19676 Command JMX Invoker

2023-06-27 Thread via GitHub
sonarcloud[bot] commented on PR #10775: URL: https://github.com/apache/ignite/pull/10775#issuecomment-1609233091 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] PakhomovAlexander commented on a diff in pull request #2219: IGNITE-19708 Check refcounter of unit before undeploy

2023-06-27 Thread via GitHub
PakhomovAlexander commented on code in PR #2219: URL: https://github.com/apache/ignite-3/pull/2219#discussion_r1243482637 ## modules/code-deployment/src/main/java/org/apache/ignite/internal/deployunit/DeploymentManagerImpl.java: ## @@ -353,6 +369,7 @@ public void start() {

[GitHub] [ignite-3] PakhomovAlexander merged pull request #2238: IGNITE-19694 Introduce deployment unit operation ID

2023-06-27 Thread via GitHub
PakhomovAlexander merged PR #2238: URL: https://github.com/apache/ignite-3/pull/2238 -- 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-3] SammyVimes commented on a diff in pull request #2249: IGNITE-19828 Optimize RAFT log encoder/decoder data format

2023-06-27 Thread via GitHub
SammyVimes commented on code in PR #2249: URL: https://github.com/apache/ignite-3/pull/2249#discussion_r1243549222 ## modules/network/src/main/java/org/apache/ignite/internal/network/direct/stream/DirectByteBufferStreamImplV1.java: ## @@ -188,7 +188,7 @@ public void setBuffer(By

[GitHub] [ignite] vshipb opened a new pull request, #10808: IGNITE-19753 merge and resolve conflicts

2023-06-27 Thread via GitHub
vshipb opened a new pull request, #10808: URL: https://github.com/apache/ignite/pull/10808 (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-3] ibessonov commented on a diff in pull request #2249: IGNITE-19828 Optimize RAFT log encoder/decoder data format

2023-06-27 Thread via GitHub
ibessonov commented on code in PR #2249: URL: https://github.com/apache/ignite-3/pull/2249#discussion_r1243598564 ## modules/raft/src/main/java/org/apache/ignite/raft/jraft/entity/codec/v1/V1Decoder.java: ## @@ -51,98 +52,127 @@ public LogEntry decode(final byte[] content) {

[GitHub] [ignite-3] ibessonov commented on a diff in pull request #2249: IGNITE-19828 Optimize RAFT log encoder/decoder data format

2023-06-27 Thread via GitHub
ibessonov commented on code in PR #2249: URL: https://github.com/apache/ignite-3/pull/2249#discussion_r1243598274 ## modules/raft/src/main/java/org/apache/ignite/raft/jraft/entity/codec/v1/V1Decoder.java: ## @@ -51,98 +52,127 @@ public LogEntry decode(final byte[] content) {

[GitHub] [ignite-3] SammyVimes commented on a diff in pull request #2249: IGNITE-19828 Optimize RAFT log encoder/decoder data format

2023-06-27 Thread via GitHub
SammyVimes commented on code in PR #2249: URL: https://github.com/apache/ignite-3/pull/2249#discussion_r1243605813 ## modules/raft/src/main/java/org/apache/ignite/raft/jraft/entity/codec/v1/V1Encoder.java: ## @@ -46,130 +48,137 @@ public byte[] encode(final LogEntry log) {

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #2261: IGNITE-19851 Add MetaStorage update update listeners and use instead of configuration revision update listeners

2023-06-27 Thread via GitHub
tkalkirill commented on code in PR #2261: URL: https://github.com/apache/ignite-3/pull/2261#discussion_r1243613403 ## modules/metastorage-api/src/main/java/org/apache/ignite/internal/metastorage/MetaStorageManager.java: ## @@ -216,4 +216,10 @@ public interface MetaStorageManager

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #2261: IGNITE-19851 Add MetaStorage update update listeners and use instead of configuration revision update listeners

2023-06-27 Thread via GitHub
tkalkirill commented on code in PR #2261: URL: https://github.com/apache/ignite-3/pull/2261#discussion_r1243613669 ## modules/metastorage/src/main/java/org/apache/ignite/internal/metastorage/impl/MetaStorageManagerImpl.java: ## @@ -671,7 +677,9 @@ private CompletableFuture onRe

[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #2261: IGNITE-19851 Add MetaStorage update update listeners and use instead of configuration revision update listeners

2023-06-27 Thread via GitHub
tkalkirill commented on code in PR #2261: URL: https://github.com/apache/ignite-3/pull/2261#discussion_r1243614330 ## modules/runner/src/main/java/org/apache/ignite/internal/app/IgniteImpl.java: ## @@ -454,7 +454,7 @@ public class IgniteImpl implements Ignite { );

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

2023-06-27 Thread via GitHub
zstan commented on PR #2241: URL: https://github.com/apache/ignite-3/pull/2241#issuecomment-1609355234 @korlov42 tests are not flaky any more, as for me code become a little bit clear for understanding. can u make a review plz ? -- This is an automated message from the Apache Git Service.

[GitHub] [ignite-3] ibessonov commented on a diff in pull request #2249: IGNITE-19828 Optimize RAFT log encoder/decoder data format

2023-06-27 Thread via GitHub
ibessonov commented on code in PR #2249: URL: https://github.com/apache/ignite-3/pull/2249#discussion_r1243621688 ## modules/raft/src/main/java/org/apache/ignite/raft/jraft/entity/codec/v1/V1Encoder.java: ## @@ -46,130 +48,137 @@ public byte[] encode(final LogEntry log) {

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10808: IGNITE-19753 merge and resolve conflicts

2023-06-27 Thread via GitHub
sonarcloud[bot] commented on PR #10808: URL: https://github.com/apache/ignite/pull/10808#issuecomment-1609369964 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] tkalkirill commented on a diff in pull request #2261: IGNITE-19851 Add MetaStorage update update listeners and use instead of configuration revision update listeners

2023-06-27 Thread via GitHub
tkalkirill commented on code in PR #2261: URL: https://github.com/apache/ignite-3/pull/2261#discussion_r1243614330 ## modules/runner/src/main/java/org/apache/ignite/internal/app/IgniteImpl.java: ## @@ -454,7 +454,7 @@ public class IgniteImpl implements Ignite { );

[GitHub] [ignite] nizhikov merged pull request #10808: IGNITE-19753 merge and resolve conflicts

2023-06-27 Thread via GitHub
nizhikov merged PR #10808: URL: https://github.com/apache/ignite/pull/10808 -- 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] sonarcloud[bot] commented on pull request #10797: IGNITE-19753 Move task classes to commands

2023-06-27 Thread via GitHub
sonarcloud[bot] commented on PR #10797: URL: https://github.com/apache/ignite/pull/10797#issuecomment-160937 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 pull request #2223: IGNITE-19668 Sql. Fix testLengthExpressionWithDynamicParameter

2023-06-27 Thread via GitHub
zstan commented on PR #2223: URL: https://github.com/apache/ignite-3/pull/2223#issuecomment-1609439821 @lowka @korlov42 new approach is ready for review, can u take a look plz ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [ignite-3] PakhomovAlexander commented on a diff in pull request #2219: IGNITE-19708 Check refcounter of unit before undeploy

2023-06-27 Thread via GitHub
PakhomovAlexander commented on code in PR #2219: URL: https://github.com/apache/ignite-3/pull/2219#discussion_r1243684104 ## modules/code-deployment/src/main/java/org/apache/ignite/internal/deployunit/DeploymentUnitAccessorImpl.java: ## @@ -36,7 +37,7 @@ public DeploymentUnitAcc

[GitHub] [ignite-3] Flaugh24 commented on a diff in pull request #2219: IGNITE-19708 Check refcounter of unit before undeploy

2023-06-27 Thread via GitHub
Flaugh24 commented on code in PR #2219: URL: https://github.com/apache/ignite-3/pull/2219#discussion_r1243690872 ## modules/code-deployment/src/main/java/org/apache/ignite/internal/deployunit/DeploymentUnitAccessorImpl.java: ## @@ -36,7 +37,7 @@ public DeploymentUnitAccessorImpl

[GitHub] [ignite-3] ibessonov merged pull request #2249: IGNITE-19828 Optimize RAFT log encoder/decoder data format

2023-06-27 Thread via GitHub
ibessonov merged PR #2249: URL: https://github.com/apache/ignite-3/pull/2249 -- 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] PakhomovAlexander commented on a diff in pull request #2219: IGNITE-19708 Check refcounter of unit before undeploy

2023-06-27 Thread via GitHub
PakhomovAlexander commented on code in PR #2219: URL: https://github.com/apache/ignite-3/pull/2219#discussion_r1243761584 ## modules/code-deployment/src/main/java/org/apache/ignite/internal/deployunit/DeploymentUnitAccessorImpl.java: ## @@ -27,6 +30,8 @@ public class Deployment

[GitHub] [ignite-3] Flaugh24 commented on a diff in pull request #2219: IGNITE-19708 Check refcounter of unit before undeploy

2023-06-27 Thread via GitHub
Flaugh24 commented on code in PR #2219: URL: https://github.com/apache/ignite-3/pull/2219#discussion_r1243765330 ## modules/code-deployment/src/main/java/org/apache/ignite/internal/deployunit/DeploymentUnitAccessorImpl.java: ## @@ -27,6 +30,8 @@ public class DeploymentUnitAcces

[GitHub] [ignite-3] Pochatkin commented on a diff in pull request #2219: IGNITE-19708 Check refcounter of unit before undeploy

2023-06-27 Thread via GitHub
Pochatkin commented on code in PR #2219: URL: https://github.com/apache/ignite-3/pull/2219#discussion_r1243585152 ## modules/code-deployment/src/test/java/org/apache/ignite/internal/deployunit/DeploymentUnitUndeployerTest.java: ## @@ -0,0 +1,140 @@ +/* + * Licensed to the Apache

[GitHub] [ignite-3] ptupitsyn merged pull request #2262: IGNITE-19610 .NET: Add data streamer metrics

2023-06-27 Thread via GitHub
ptupitsyn merged PR #2262: URL: https://github.com/apache/ignite-3/pull/2262 -- 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] rpuch opened a new pull request, #2264: IGNITE-19857 Default zone replica count is not listened by rebalance trigger

2023-06-27 Thread via GitHub
rpuch opened a new pull request, #2264: URL: https://github.com/apache/ignite-3/pull/2264 https://issues.apache.org/jira/browse/IGNITE-19857 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the sp

[GitHub] [ignite-3] PakhomovAlexander merged pull request #2219: IGNITE-19708 Check refcounter of unit before undeploy

2023-06-27 Thread via GitHub
PakhomovAlexander merged PR #2219: URL: https://github.com/apache/ignite-3/pull/2219 -- 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-3] sanpwc commented on a diff in pull request #2253: IGNITE-19580 added restoring of zones state after restart

2023-06-27 Thread via GitHub
sanpwc commented on code in PR #2253: URL: https://github.com/apache/ignite-3/pull/2253#discussion_r1243957628 ## modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DistributionZoneManager.java: ## @@ -798,15 +787,81 @@ public CompletableFuture

[GitHub] [ignite-3] sanpwc commented on a diff in pull request #2253: IGNITE-19580 added restoring of zones state after restart

2023-06-27 Thread via GitHub
sanpwc commented on code in PR #2253: URL: https://github.com/apache/ignite-3/pull/2253#discussion_r1243972544 ## modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DistributionZoneManager.java: ## @@ -798,15 +787,81 @@ public CompletableFuture

[GitHub] [ignite-3] sanpwc commented on a diff in pull request #2253: IGNITE-19580 added restoring of zones state after restart

2023-06-27 Thread via GitHub
sanpwc commented on code in PR #2253: URL: https://github.com/apache/ignite-3/pull/2253#discussion_r1243972544 ## modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DistributionZoneManager.java: ## @@ -798,15 +787,81 @@ public CompletableFuture

[GitHub] [ignite-3] sanpwc commented on a diff in pull request #2253: IGNITE-19580 added restoring of zones state after restart

2023-06-27 Thread via GitHub
sanpwc commented on code in PR #2253: URL: https://github.com/apache/ignite-3/pull/2253#discussion_r1244005904 ## modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DistributionZoneManager.java: ## @@ -854,7 +920,7 @@ private void removeTrigger

[GitHub] [ignite-3] sanpwc commented on a diff in pull request #2253: IGNITE-19580 added restoring of zones state after restart

2023-06-27 Thread via GitHub
sanpwc commented on code in PR #2253: URL: https://github.com/apache/ignite-3/pull/2253#discussion_r1244012838 ## modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DistributionZonesUtil.java: ## @@ -250,24 +260,11 @@ static CompoundCondition

[GitHub] [ignite-3] sanpwc commented on a diff in pull request #2253: IGNITE-19580 added restoring of zones state after restart

2023-06-27 Thread via GitHub
sanpwc commented on code in PR #2253: URL: https://github.com/apache/ignite-3/pull/2253#discussion_r1244012838 ## modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DistributionZonesUtil.java: ## @@ -250,24 +260,11 @@ static CompoundCondition

[GitHub] [ignite-3] alievmirza commented on a diff in pull request #2255: IGNITE-19120 Raft client should get leader metadata along while getting leader itself

2023-06-27 Thread via GitHub
alievmirza commented on code in PR #2255: URL: https://github.com/apache/ignite-3/pull/2255#discussion_r1243939875 ## modules/raft/src/main/java/org/apache/ignite/raft/jraft/Node.java: ## @@ -41,6 +42,14 @@ public interface Node extends Lifecycle, Describer { */ Peer

[GitHub] [ignite-3] sk0x50 merged pull request #2252: IGNITE-19724 Removed redundant joins

2023-06-27 Thread via GitHub
sk0x50 merged PR #2252: URL: https://github.com/apache/ignite-3/pull/2252 -- 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] vshipb opened a new pull request, #10809: IGNITE-19753 merge master

2023-06-27 Thread via GitHub
vshipb opened a new pull request, #10809: URL: https://github.com/apache/ignite/pull/10809 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: ### The

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10809: IGNITE-19753 merge master

2023-06-27 Thread via GitHub
sonarcloud[bot] commented on PR #10809: URL: https://github.com/apache/ignite/pull/10809#issuecomment-1610018147 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] AMashenkov opened a new pull request, #2265: IGNITE-19775 Add index builder to TestBuilders

2023-06-27 Thread via GitHub
AMashenkov opened a new pull request, #2265: URL: https://github.com/apache/ignite-3/pull/2265 https://issues.apache.org/jira/browse/IGNITE-19775 -- 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] korlov42 commented on a diff in pull request #2241: IGNITE-19730 Sql. ExecutionServiceImplTest different tests failed

2023-06-27 Thread via GitHub
korlov42 commented on code in PR #2241: URL: https://github.com/apache/ignite-3/pull/2241#discussion_r1244714682 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/sql/api/ItSqlAsynchronousApiTest.java: ## @@ -690,7 +689,7 @@ public void closeSession() {

[GitHub] [ignite] Indupa opened a new issue, #10810: Apache Ignite 2.15 startup failure with JDK 17 on windows server

2023-06-28 Thread via GitHub
Indupa opened a new issue, #10810: URL: https://github.com/apache/ignite/issues/10810 Hi , In our project , ignite is started using "Topshelf" command to run as windows service by passing JAVA_HOME as one of the Argument. when i tried to run Apche Ignite 2.15.0 with OpenJDK 17, Its

[GitHub] [ignite-3] sanpwc commented on a diff in pull request #2253: IGNITE-19580 added restoring of zones state after restart

2023-06-28 Thread via GitHub
sanpwc commented on code in PR #2253: URL: https://github.com/apache/ignite-3/pull/2253#discussion_r1244781559 ## modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DistributionZoneManager.java: ## @@ -798,15 +787,81 @@ public CompletableFuture

[GitHub] [ignite] ptupitsyn commented on issue #10810: Apache Ignite 2.15 startup failure with JDK 17 on windows server

2023-06-28 Thread via GitHub
ptupitsyn commented on issue #10810: URL: https://github.com/apache/ignite/issues/10810#issuecomment-1610899797 Please do not create duplicate issues: [10747](https://github.com/apache/ignite/issues/10747) -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [ignite] ptupitsyn closed issue #10810: Apache Ignite 2.15 startup failure with JDK 17 on windows server

2023-06-28 Thread via GitHub
ptupitsyn closed issue #10810: Apache Ignite 2.15 startup failure with JDK 17 on windows server URL: https://github.com/apache/ignite/issues/10810 -- 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

[GitHub] [ignite-3] zstan commented on a diff in pull request #2241: IGNITE-19730 Sql. ExecutionServiceImplTest different tests failed

2023-06-28 Thread via GitHub
zstan commented on code in PR #2241: URL: https://github.com/apache/ignite-3/pull/2241#discussion_r1244802526 ## modules/runner/src/integrationTest/java/org/apache/ignite/internal/sql/api/ItSqlAsynchronousApiTest.java: ## @@ -690,7 +689,7 @@ public void closeSession() {

[GitHub] [ignite-3] sanpwc commented on a diff in pull request #2253: IGNITE-19580 added restoring of zones state after restart

2023-06-28 Thread via GitHub
sanpwc commented on code in PR #2253: URL: https://github.com/apache/ignite-3/pull/2253#discussion_r1244814330 ## modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DistributionZoneManager.java: ## @@ -798,15 +787,85 @@ public CompletableFuture

[GitHub] [ignite-3] alievmirza commented on a diff in pull request #2253: IGNITE-19580 added restoring of zones state after restart

2023-06-28 Thread via GitHub
alievmirza commented on code in PR #2253: URL: https://github.com/apache/ignite-3/pull/2253#discussion_r1244830911 ## modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DistributionZoneManager.java: ## @@ -798,15 +787,85 @@ public CompletableFu

[GitHub] [ignite-3] alievmirza commented on a diff in pull request #2253: IGNITE-19580 added restoring of zones state after restart

2023-06-28 Thread via GitHub
alievmirza commented on code in PR #2253: URL: https://github.com/apache/ignite-3/pull/2253#discussion_r1244831217 ## modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DistributionZoneManager.java: ## @@ -854,7 +920,7 @@ private void removeTri

[GitHub] [ignite-3] zstan commented on a diff in pull request #2241: IGNITE-19730 Sql. ExecutionServiceImplTest different tests failed

2023-06-28 Thread via GitHub
zstan commented on code in PR #2241: URL: https://github.com/apache/ignite-3/pull/2241#discussion_r1244838075 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/ExecutionServiceImpl.java: ## @@ -888,14 +903,6 @@ private CompletableFuture awaitFragmen

[GitHub] [ignite-3] zstan commented on a diff in pull request #2241: IGNITE-19730 Sql. ExecutionServiceImplTest different tests failed

2023-06-28 Thread via GitHub
zstan commented on code in PR #2241: URL: https://github.com/apache/ignite-3/pull/2241#discussion_r1244838075 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/ExecutionServiceImpl.java: ## @@ -888,14 +903,6 @@ private CompletableFuture awaitFragmen

[GitHub] [ignite] nizhikov merged pull request #10809: IGNITE-19753 merge master

2023-06-28 Thread via GitHub
nizhikov merged PR #10809: URL: https://github.com/apache/ignite/pull/10809 -- 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-3] rpuch commented on a diff in pull request #2261: IGNITE-19851 Add MetaStorage revision update listeners and use instead of configuration revision update listeners

2023-06-28 Thread via GitHub
rpuch commented on code in PR #2261: URL: https://github.com/apache/ignite-3/pull/2261#discussion_r1244805715 ## modules/configuration/src/main/java/org/apache/ignite/internal/configuration/notifications/ConfigurationStorageRevisionListener.java: ## @@ -24,6 +24,7 @@ * * Sto

[GitHub] [ignite-3] rpuch commented on a diff in pull request #2261: IGNITE-19851 Add MetaStorage revision update listeners and use instead of configuration revision update listeners

2023-06-28 Thread via GitHub
rpuch commented on code in PR #2261: URL: https://github.com/apache/ignite-3/pull/2261#discussion_r1244842192 ## modules/metastorage/src/main/java/org/apache/ignite/internal/metastorage/server/WatchProcessor.java: ## @@ -158,7 +158,7 @@ private static CompletableFuture notifyWa

[GitHub] [ignite-3] korlov42 commented on a diff in pull request #2241: IGNITE-19730 Sql. ExecutionServiceImplTest different tests failed

2023-06-28 Thread via GitHub
korlov42 commented on code in PR #2241: URL: https://github.com/apache/ignite-3/pull/2241#discussion_r1244846036 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/ExecutionServiceImpl.java: ## @@ -888,14 +903,6 @@ private CompletableFuture awaitFrag

[GitHub] [ignite-3] ibessonov commented on a diff in pull request #2261: IGNITE-19851 Add MetaStorage revision update listeners and use instead of configuration revision update listeners

2023-06-28 Thread via GitHub
ibessonov commented on code in PR #2261: URL: https://github.com/apache/ignite-3/pull/2261#discussion_r1244848815 ## modules/metastorage-api/src/main/java/org/apache/ignite/internal/metastorage/MetaStorageManager.java: ## @@ -222,4 +222,13 @@ public interface MetaStorageManager

[GitHub] [ignite-3] ibessonov commented on a diff in pull request #2261: IGNITE-19851 Add MetaStorage revision update listeners and use instead of configuration revision update listeners

2023-06-28 Thread via GitHub
ibessonov commented on code in PR #2261: URL: https://github.com/apache/ignite-3/pull/2261#discussion_r1244852795 ## modules/metastorage-api/src/main/java/org/apache/ignite/internal/metastorage/RevisionUpdateListener.java: ## @@ -0,0 +1,34 @@ +/* + * Licensed to the Apache Softw

[GitHub] [ignite] sonarcloud[bot] commented on pull request #10797: IGNITE-19753 Move task classes to commands

2023-06-28 Thread via GitHub
sonarcloud[bot] commented on PR #10797: URL: https://github.com/apache/ignite/pull/10797#issuecomment-1610967408 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 #2261: IGNITE-19851 Add MetaStorage revision update listeners and use instead of configuration revision update listeners

2023-06-28 Thread via GitHub
ibessonov commented on code in PR #2261: URL: https://github.com/apache/ignite-3/pull/2261#discussion_r1244856173 ## modules/metastorage/src/integrationTest/java/org/apache/ignite/internal/metastorage/impl/ItMetaStorageManagerImplTest.java: ## @@ -284,4 +286,24 @@ void testMetaS

[GitHub] [ignite-3] sashapolo commented on a diff in pull request #2239: IGNITE-19199 Propagate safe time when Meta Storage is idle

2023-06-28 Thread via GitHub
sashapolo commented on code in PR #2239: URL: https://github.com/apache/ignite-3/pull/2239#discussion_r1244858734 ## modules/metastorage/src/main/java/org/apache/ignite/internal/metastorage/impl/MetaStorageManagerImpl.java: ## @@ -137,66 +144,123 @@ public MetaStorageManagerImpl

[GitHub] [ignite-3] sashapolo commented on a diff in pull request #2239: IGNITE-19199 Propagate safe time when Meta Storage is idle

2023-06-28 Thread via GitHub
sashapolo commented on code in PR #2239: URL: https://github.com/apache/ignite-3/pull/2239#discussion_r1244860586 ## modules/metastorage/src/main/java/org/apache/ignite/internal/metastorage/impl/MetaStorageManagerImpl.java: ## @@ -137,66 +144,123 @@ public MetaStorageManagerImpl

<    3   4   5   6   7   8   9   10   11   12   >