Re: [PR] IGNITE-24913 WIP [ignite]

2025-03-27 Thread via GitHub
petrov-mg closed pull request #11971: IGNITE-24913 WIP URL: https://github.com/apache/ignite/pull/11971 -- 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:

Re: [PR] IGNITE-24913 Fixed flaky TcpCommunicationSpiSslVolatilePayloadTest test. [ignite]

2025-03-27 Thread via GitHub
petrov-mg merged PR #11962: URL: https://github.com/apache/ignite/pull/11962 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@i

[PR] IGNITE-24862 Rebuild ignite 2.16 doc [ignite-website]

2025-03-27 Thread via GitHub
IgGusev opened a new pull request, #219: URL: https://github.com/apache/ignite-website/pull/219 (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,

Re: [PR] IGNITE-24535 Compatibility test for Ignite snapshot [ignite]

2025-03-27 Thread via GitHub
nizhikov commented on code in PR #11939: URL: https://github.com/apache/ignite/pull/11939#discussion_r2017498678 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -0,0 +1,458 @@ +/* + * Licensed to the Apache

[PR] IGNITE-24935 Fixed node failure with CancelledKeyException on session close with SSL enabled. [ignite]

2025-03-27 Thread via GitHub
petrov-mg opened a new pull request, #11973: URL: https://github.com/apache/ignite/pull/11973 Thank you for submitting the pull request to the Apache Ignite. In order to streamline the review of the contribution we ask you to ensure the following steps have been taken: ### T

Re: [PR] IGNITE-24535 Compatibility test for Ignite snapshot [ignite]

2025-03-27 Thread via GitHub
nizhikov commented on code in PR #11939: URL: https://github.com/apache/ignite/pull/11939#discussion_r2017495686 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -0,0 +1,458 @@ +/* + * Licensed to the Apache

Re: [PR] IGNITE-24938 Ignore attempts to create index storage in destroyed partition [ignite-3]

2025-03-27 Thread via GitHub
rpuch merged PR #5509: URL: https://github.com/apache/ignite-3/pull/5509 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@ignit

Re: [PR] IGNITE-24753 SQL Calcite: Fix freeze in MergeJoin right/full execution nodes [ignite]

2025-03-27 Thread via GitHub
Vladsz83 commented on code in PR #11955: URL: https://github.com/apache/ignite/pull/11955#discussion_r2016840788 ## modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/integration/tpch/TpchTest.java: ## @@ -17,22 +17,47 @@ package org.apache.igni

Re: [PR] IGNITE-24884 : Calcite. Limit joins number to commute in planning . [ignite]

2025-03-27 Thread via GitHub
alex-plekhanov commented on code in PR #11957: URL: https://github.com/apache/ignite/pull/11957#discussion_r2017482987 ## modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/planner/JoinCommutePlannerTest.java: ## @@ -76,6 +86,105 @@ public class Jo

[PR] IGNITE-24913 WIP [ignite]

2025-03-27 Thread via GitHub
petrov-mg opened a new pull request, #11971: URL: https://github.com/apache/ignite/pull/11971 Thank you for submitting the pull request to the Apache Ignite. In order to streamline the review of the contribution we ask you to ensure the following steps have been taken: ### T

Re: [PR] IGNITE-24884 : Calcite. Limit joins number to commute in planning . [ignite]

2025-03-27 Thread via GitHub
Vladsz83 commented on code in PR #11957: URL: https://github.com/apache/ignite/pull/11957#discussion_r2017494550 ## modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/planner/JoinCommutePlannerTest.java: ## @@ -76,6 +86,105 @@ public class JoinComm

Re: [PR] IGNITE-24535 Compatibility test for Ignite snapshot [ignite]

2025-03-27 Thread via GitHub
nizhikov commented on code in PR #11939: URL: https://github.com/apache/ignite/pull/11939#discussion_r2017494568 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -0,0 +1,458 @@ +/* + * Licensed to the Apache

Re: [PR] IGNITE-24535 Compatibility test for Ignite snapshot [ignite]

2025-03-27 Thread via GitHub
nizhikov commented on code in PR #11939: URL: https://github.com/apache/ignite/pull/11939#discussion_r2017494791 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -0,0 +1,458 @@ +/* + * Licensed to the Apache

Re: [PR] IGNITE-24535 Compatibility test for Ignite snapshot [ignite]

2025-03-27 Thread via GitHub
nizhikov commented on code in PR #11939: URL: https://github.com/apache/ignite/pull/11939#discussion_r2017493662 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -0,0 +1,458 @@ +/* + * Licensed to the Apache

Re: [PR] IGNITE-24535 Compatibility test for Ignite snapshot [ignite]

2025-03-27 Thread via GitHub
nizhikov commented on code in PR #11939: URL: https://github.com/apache/ignite/pull/11939#discussion_r2017495092 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -0,0 +1,458 @@ +/* + * Licensed to the Apache

Re: [PR] IGNITE-24535 Compatibility test for Ignite snapshot [ignite]

2025-03-27 Thread via GitHub
nizhikov commented on code in PR #11939: URL: https://github.com/apache/ignite/pull/11939#discussion_r2017496939 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -0,0 +1,458 @@ +/* + * Licensed to the Apache

Re: [PR] IGNITE-24535 Compatibility test for Ignite snapshot [ignite]

2025-03-27 Thread via GitHub
nizhikov commented on code in PR #11939: URL: https://github.com/apache/ignite/pull/11939#discussion_r2017496364 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -0,0 +1,458 @@ +/* + * Licensed to the Apache

Re: [PR] IGNITE-24535 Compatibility test for Ignite snapshot [ignite]

2025-03-27 Thread via GitHub
nizhikov commented on code in PR #11939: URL: https://github.com/apache/ignite/pull/11939#discussion_r2017497194 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -0,0 +1,458 @@ +/* + * Licensed to the Apache

Re: [PR] IGNITE-24535 Compatibility test for Ignite snapshot [ignite]

2025-03-27 Thread via GitHub
nizhikov commented on code in PR #11939: URL: https://github.com/apache/ignite/pull/11939#discussion_r2017499121 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -0,0 +1,458 @@ +/* + * Licensed to the Apache

Re: [PR] IGNITE-24884 : Calcite. Limit joins number to commute in planning . [ignite]

2025-03-27 Thread via GitHub
Vladsz83 commented on code in PR #11957: URL: https://github.com/apache/ignite/pull/11957#discussion_r2017494550 ## modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/planner/JoinCommutePlannerTest.java: ## @@ -76,6 +86,105 @@ public class JoinComm

[PR] IGNITE-24923 Sql. Rework execution to process batch at a time [ignite-3]

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

Re: [PR] IGNITE-23919 Remove DurableBackgroundCleanupIndexTreeTask and related code [ignite]

2025-03-27 Thread via GitHub
nizhikov merged PR #11959: URL: https://github.com/apache/ignite/pull/11959 -- 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

[PR] IGNITE-24779 Support index lifecycle for per-zone partitions [ignite-3]

2025-03-27 Thread via GitHub
rpuch opened a new pull request, #5431: URL: https://github.com/apache/ignite-3/pull/5431 https://issues.apache.org/jira/browse/IGNITE-24779 * ChangeIndexStatusTaskController now supports primacy tracking for per-zone partitions * Do not try to start building an index created with

Re: [PR] Bump com.google.guava:guava from 33.4.5-jre to 33.4.6-jre [ignite-3]

2025-03-27 Thread via GitHub
ptupitsyn merged PR #5503: URL: https://github.com/apache/ignite-3/pull/5503 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@i

Re: [PR] IGNITE-24535 Compatibility test for Ignite snapshot [ignite]

2025-03-27 Thread via GitHub
nizhikov commented on code in PR #11939: URL: https://github.com/apache/ignite/pull/11939#discussion_r2016113768 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -0,0 +1,452 @@ +/* + * Licensed to the Apache

Re: [PR] IGNITE-24535 Compatibility test for Ignite snapshot [ignite]

2025-03-27 Thread via GitHub
nizhikov commented on code in PR #11939: URL: https://github.com/apache/ignite/pull/11939#discussion_r2016115358 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -0,0 +1,452 @@ +/* + * Licensed to the Apache

Re: [PR] IGNITE-24535 Compatibility test for Ignite snapshot [ignite]

2025-03-27 Thread via GitHub
nizhikov commented on code in PR #11939: URL: https://github.com/apache/ignite/pull/11939#discussion_r2016119747 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -0,0 +1,452 @@ +/* + * Licensed to the Apache

Re: [PR] IGNITE-24588 Fixed client-side tuple serialization when the schema has field names containing non-standard characters. [ignite-3]

2025-03-27 Thread via GitHub
AMashenkov commented on code in PR #5493: URL: https://github.com/apache/ignite-3/pull/5493#discussion_r2016139761 ## modules/table/src/integrationTest/java/org/apache/ignite/internal/table/ItTableViewApiUnifiedBaseTest.java: ## @@ -82,17 +83,69 @@ protected int initialNodes() {

Re: [PR] IGNITE-24535 Compatibility test for Ignite snapshot [ignite]

2025-03-27 Thread via GitHub
nizhikov commented on code in PR #11939: URL: https://github.com/apache/ignite/pull/11939#discussion_r2016140503 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -0,0 +1,452 @@ +/* + * Licensed to the Apache

Re: [PR] IGNITE-24935 Switch MvTableStorage to create*Index methods [ignite-3]

2025-03-27 Thread via GitHub
rpuch merged PR #5506: URL: https://github.com/apache/ignite-3/pull/5506 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@ignit

Re: [PR] IGNITE-24588 Fixed client-side tuple serialization when the schema has field names containing non-standard characters. [ignite-3]

2025-03-27 Thread via GitHub
AMashenkov commented on code in PR #5493: URL: https://github.com/apache/ignite-3/pull/5493#discussion_r2016131595 ## modules/table/src/integrationTest/java/org/apache/ignite/internal/table/ItKeyValueViewSimpleSchemaApiTest.java: ## @@ -60,28 +60,38 @@ public class ItKeyValueVie

Re: [PR] IGNITE-24552: Failure upon creating a table with default TIMESTAMP column [ignite-3]

2025-03-27 Thread via GitHub
korlov42 commented on code in PR #5450: URL: https://github.com/apache/ignite-3/pull/5450#discussion_r2016117054 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/prepare/ddl/DdlSqlToCommandConverter.java: ## @@ -562,25 +555,40 @@ private CatalogCommand

[PR] IGNITE-24936 Sql. Rename virtual column "__part" to allow access without quoting [ignite-3]

2025-03-27 Thread via GitHub
korlov42 opened a new pull request, #5507: URL: https://github.com/apache/ignite-3/pull/5507 https://issues.apache.org/jira/browse/IGNITE-24936 - Thank you for submitting the pull request. To streamline the review process of the patch and ensure better code quality

Re: [PR] IGNITE-24588 Fixed client-side tuple serialization when the schema has field names containing non-standard characters. [ignite-3]

2025-03-27 Thread via GitHub
AMashenkov commented on code in PR #5493: URL: https://github.com/apache/ignite-3/pull/5493#discussion_r2016153714 ## modules/table/src/integrationTest/java/org/apache/ignite/internal/table/ItTableViewApiUnifiedBaseTest.java: ## @@ -82,17 +83,69 @@ protected int initialNodes() {

Re: [PR] IGNITE-24552: Failure upon creating a table with default TIMESTAMP column [ignite-3]

2025-03-27 Thread via GitHub
lowka commented on code in PR #5450: URL: https://github.com/apache/ignite-3/pull/5450#discussion_r2016145654 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/prepare/ddl/DdlSqlToCommandConverter.java: ## @@ -562,25 +555,40 @@ private CatalogCommand con

Re: [PR] IGNITE-24835 Forbid use of TIMESTAMP_TZ at parser level. [ignite-3]

2025-03-27 Thread via GitHub
xtern commented on code in PR #5490: URL: https://github.com/apache/ignite-3/pull/5490#discussion_r2015722777 ## modules/sql-engine/src/main/codegen/includes/parserImpls.ftl: ## @@ -675,19 +675,32 @@ SqlTypeNameSpec IgniteDateTimeTypeName() : | { s = span(); } precis

Re: [PR] IGNITE-23167 Fixed warning about using KEY in MERGE queries. [ignite]

2025-03-27 Thread via GitHub
sonarqubecloud[bot] commented on PR #11967: URL: https://github.com/apache/ignite/pull/11967#issuecomment-2758283810 ## [![Quality Gate Failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-failed-20px.png 'Quality Gate Failed')](https://son

[PR] IGNITE-24756 SQL Calcite: Add extraction of common part from disjunct… [ignite]

2025-03-27 Thread via GitHub
alex-plekhanov opened a new pull request, #11972: URL: https://github.com/apache/ignite/pull/11972 …ion in conditions 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 step

Re: [PR] IGNITE-24756 SQL Calcite: Add extraction of common part from disjunct… [ignite]

2025-03-27 Thread via GitHub
sonarqubecloud[bot] commented on PR #11972: URL: https://github.com/apache/ignite/pull/11972#issuecomment-2759067440 ## [![Quality Gate Failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-failed-20px.png 'Quality Gate Failed')](https://son

Re: [PR] IGNITE-23856 channel connection retry in SYNC case [ignite]

2025-03-27 Thread via GitHub
sonarqubecloud[bot] commented on PR #11953: URL: https://github.com/apache/ignite/pull/11953#issuecomment-2759072323 ## [![Quality Gate Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png 'Quality Gate Passed')](https://son

Re: [PR] IGNITE-23856 channel connection retry in SYNC case [ignite]

2025-03-27 Thread via GitHub
timoninmaxim merged PR #11953: URL: https://github.com/apache/ignite/pull/11953 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr..

[PR] IGNITE-24944 Improve TrackerClosedException handling [ignite-3]

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

Re: [PR] IGNITE-24552: Failure upon creating a table with default TIMESTAMP column [ignite-3]

2025-03-27 Thread via GitHub
lowka commented on code in PR #5450: URL: https://github.com/apache/ignite-3/pull/5450#discussion_r2016862672 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/prepare/ddl/DdlSqlToCommandConverter.java: ## @@ -562,25 +555,40 @@ private CatalogCommand con

Re: [PR] IGNITE-24913 WIP [ignite]

2025-03-27 Thread via GitHub
sonarqubecloud[bot] commented on PR #11971: URL: https://github.com/apache/ignite/pull/11971#issuecomment-2758845156 ## [![Quality Gate Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png 'Quality Gate Passed')](https://son

Re: [PR] IGNITE-24588 Fixed client-side tuple serialization when the schema has field names containing non-standard characters. [ignite-3]

2025-03-27 Thread via GitHub
AMashenkov commented on code in PR #5493: URL: https://github.com/apache/ignite-3/pull/5493#discussion_r2016150998 ## modules/table/src/integrationTest/java/org/apache/ignite/internal/table/ItTableViewApiUnifiedBaseTest.java: ## @@ -82,17 +83,69 @@ protected int initialNodes() {

Re: [PR] IGNITE-24884 : Calcite. Limit joins number to commute in planning . [ignite]

2025-03-27 Thread via GitHub
alex-plekhanov commented on code in PR #11957: URL: https://github.com/apache/ignite/pull/11957#discussion_r2017249623 ## modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/planner/JoinCommutePlannerTest.java: ## @@ -76,6 +82,100 @@ public class Jo

Re: [PR] IGNITE-24884 : Calcite. Limit joins number to commute in planning . [ignite]

2025-03-27 Thread via GitHub
Vladsz83 commented on code in PR #11957: URL: https://github.com/apache/ignite/pull/11957#discussion_r2017317034 ## modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/planner/JoinCommutePlannerTest.java: ## @@ -76,6 +82,100 @@ public class JoinComm

Re: [PR] IGNITE-24884 : Calcite. Limit joins number to commute in planning . [ignite]

2025-03-27 Thread via GitHub
sonarqubecloud[bot] commented on PR #11957: URL: https://github.com/apache/ignite/pull/11957#issuecomment-2759327324 ## [![Quality Gate Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png 'Quality Gate Passed')](https://son

Re: [PR] IGNITE-24935 Fixed node failure with CancelledKeyException on session close with SSL enabled. [ignite]

2025-03-27 Thread via GitHub
sonarqubecloud[bot] commented on PR #11973: URL: https://github.com/apache/ignite/pull/11973#issuecomment-2759440861 ## [![Quality Gate Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png 'Quality Gate Passed')](https://son

Re: [PR] IGNITE-19528 CDC examples with cdc-start-up.sh [ignite-extensions]

2025-03-27 Thread via GitHub
timoninmaxim merged PR #285: URL: https://github.com/apache/ignite-extensions/pull/285 -- 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-uns

Re: [PR] IGNITE-24535 Compatibility test for Ignite snapshot [ignite]

2025-03-27 Thread via GitHub
nizhikov commented on code in PR #11939: URL: https://github.com/apache/ignite/pull/11939#discussion_r2016112388 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -0,0 +1,452 @@ +/* + * Licensed to the Apache

[PR] Bump com.google.protobuf:protobuf-java from 4.30.1 to 4.30.2 [ignite-3]

2025-03-27 Thread via GitHub
dependabot[bot] opened a new pull request, #5511: URL: https://github.com/apache/ignite-3/pull/5511 Bumps [com.google.protobuf:protobuf-java](https://github.com/protocolbuffers/protobuf) from 4.30.1 to 4.30.2. Commits https://github.com/protocolbuffers/protobuf/commit/43e16268

Re: [PR] IGNITE-23167 Fixed warning about using KEY in MERGE queries. [ignite]

2025-03-27 Thread via GitHub
NSAmelchev merged PR #11967: URL: https://github.com/apache/ignite/pull/11967 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@

Re: [PR] Ignite 23167 tc debug [ignite]

2025-03-27 Thread via GitHub
sonarqubecloud[bot] commented on PR #11967: URL: https://github.com/apache/ignite/pull/11967#issuecomment-2757570327 ## [![Quality Gate Failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-failed-20px.png 'Quality Gate Failed')](https://son

Re: [PR] IGNITE-24588 Fixed client-side tuple serialization when the schema has field names containing non-standard characters. [ignite-3]

2025-03-27 Thread via GitHub
xtern commented on code in PR #5493: URL: https://github.com/apache/ignite-3/pull/5493#discussion_r2016447334 ## modules/table/src/integrationTest/java/org/apache/ignite/internal/table/ItKeyValueViewSimpleSchemaApiTest.java: ## @@ -60,28 +60,38 @@ public class ItKeyValueViewSimp

[PR] IGNITE-24938 Ignore attempts to create index storage in destroyed partition [ignite-3]

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

Re: [PR] IGNITE-24835 Forbid use of TIMESTAMP_TZ at parser level. [ignite-3]

2025-03-27 Thread via GitHub
xtern merged PR #5490: URL: https://github.com/apache/ignite-3/pull/5490 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@ignit

Re: [PR] IGNITE-24586 Tuple API. Add methods to reading byte array values [ignite-3]

2025-03-27 Thread via GitHub
ygerzhedovich commented on code in PR #5486: URL: https://github.com/apache/ignite-3/pull/5486#discussion_r2016443566 ## modules/api/src/testFixtures/java/org/apache/ignite/table/AbstractImmutableTupleTest.java: ## @@ -352,4 +378,66 @@ private static int tailFactor(int precision

Re: [PR] IGNITE-24786 Cleanup GridFunc [ignite]

2025-03-27 Thread via GitHub
sonarqubecloud[bot] commented on PR #11944: URL: https://github.com/apache/ignite/pull/11944#issuecomment-2757846243 ## [![Quality Gate Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png 'Quality Gate Passed')](https://son

Re: [PR] IGNITE-24936 Sql. Rename virtual column "__part" to allow access without quoting [ignite-3]

2025-03-27 Thread via GitHub
korlov42 closed pull request #5507: IGNITE-24936 Sql. Rename virtual column "__part" to allow access without quoting URL: https://github.com/apache/ignite-3/pull/5507 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [PR] IGNITE-24636 Sql. Fix freezes in MergeJoin[type=RIGHT] [ignite-3]

2025-03-27 Thread via GitHub
Vladsz83 commented on code in PR #5294: URL: https://github.com/apache/ignite-3/pull/5294#discussion_r2016484576 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/rel/MergeJoinNode.java: ## @@ -989,10 +991,12 @@ protected void join() throws Exception

Re: [PR] IGNITE-24928 Validate that node finder type is correct [ignite-3]

2025-03-27 Thread via GitHub
tkalkirill commented on code in PR #5505: URL: https://github.com/apache/ignite-3/pull/5505#discussion_r2015847236 ## modules/network/src/main/java/org/apache/ignite/internal/network/configuration/MulticastConfigurationSchema.java: ## @@ -42,8 +42,8 @@ public class MulticastConf

Re: [PR] IGNITE-20392 Move cassandra modules to extensions [ignite-extensions]

2025-03-27 Thread via GitHub
rolf-rook commented on PR #233: URL: https://github.com/apache/ignite-extensions/pull/233#issuecomment-2758065683 When will the Cassandra modules be available as an Ignite extension? Remark that the Cassandra modules are removed from Ignite since release 2.16 -- This is an automated mess

Re: [PR] IGNITE-24938 Ignore attempts to create index storage in destroyed partition [ignite-3]

2025-03-27 Thread via GitHub
rpuch commented on code in PR #5509: URL: https://github.com/apache/ignite-3/pull/5509#discussion_r2016543973 ## modules/index/src/integrationTest/java/org/apache/ignite/internal/index/ItIndexRegistrationAndRebalanceConcurrencyTest.java: ## @@ -0,0 +1,61 @@ +/* + * Licensed to t

Re: [PR] IGNITE-24938 Ignore attempts to create index storage in destroyed partition [ignite-3]

2025-03-27 Thread via GitHub
rpuch commented on code in PR #5509: URL: https://github.com/apache/ignite-3/pull/5509#discussion_r2016562483 ## modules/table/src/main/java/org/apache/ignite/internal/table/distributed/TableManager.java: ## @@ -2883,6 +2879,10 @@ private CompletableFuture stopTablePartition(Ta

Re: [PR] IGNITE-24636 Sql. Fix freezes in MergeJoin[type=RIGHT] [ignite-3]

2025-03-27 Thread via GitHub
Vladsz83 commented on code in PR #5294: URL: https://github.com/apache/ignite-3/pull/5294#discussion_r2016484576 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/rel/MergeJoinNode.java: ## @@ -989,10 +991,12 @@ protected void join() throws Exception

Re: [PR] IGNITE-24753 SQL Calcite: Fix freeze in MergeJoin right/full execution nodes [ignite]

2025-03-27 Thread via GitHub
Vladsz83 commented on code in PR #11955: URL: https://github.com/apache/ignite/pull/11955#discussion_r2016574914 ## modules/calcite/src/main/java/org/apache/ignite/internal/processors/query/calcite/exec/rel/MergeJoinNode.java: ## @@ -962,10 +965,13 @@ else if (cmp > 0) {

Re: [PR] IGNITE-24938 Ignore attempts to create index storage in destroyed partition [ignite-3]

2025-03-27 Thread via GitHub
sashapolo commented on code in PR #5509: URL: https://github.com/apache/ignite-3/pull/5509#discussion_r2016586153 ## modules/index/src/integrationTest/java/org/apache/ignite/internal/index/ItIndexRegistrationAndRebalanceConcurrencyTest.java: ## @@ -0,0 +1,61 @@ +/* + * Licensed

Re: [PR] IGNITE-24786 Cleanup GridFunc [ignite]

2025-03-27 Thread via GitHub
sonarqubecloud[bot] commented on PR #11944: URL: https://github.com/apache/ignite/pull/11944#issuecomment-2757902315 ## [![Quality Gate Failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-failed-20px.png 'Quality Gate Failed')](https://son

Re: [PR] IGNITE-24786 Cleanup GridFunc [ignite]

2025-03-27 Thread via GitHub
sonarqubecloud[bot] commented on PR #11944: URL: https://github.com/apache/ignite/pull/11944#issuecomment-2757927881 ## [![Quality Gate Failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-failed-20px.png 'Quality Gate Failed')](https://son

Re: [PR] IGNITE-24938 Ignore attempts to create index storage in destroyed partition [ignite-3]

2025-03-27 Thread via GitHub
sashapolo commented on code in PR #5509: URL: https://github.com/apache/ignite-3/pull/5509#discussion_r2016527743 ## modules/index/src/integrationTest/java/org/apache/ignite/internal/index/ItIndexRegistrationAndRebalanceConcurrencyTest.java: ## @@ -0,0 +1,61 @@ +/* + * Licensed

Re: [PR] IGNITE-24535 Compatibility test for Ignite snapshot [ignite]

2025-03-27 Thread via GitHub
nizhikov commented on code in PR #11939: URL: https://github.com/apache/ignite/pull/11939#discussion_r2016111419 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -0,0 +1,452 @@ +/* + * Licensed to the Apache

Re: [PR] IGNITE-24938 Ignore attempts to create index storage in destroyed partition [ignite-3]

2025-03-27 Thread via GitHub
sashapolo commented on code in PR #5509: URL: https://github.com/apache/ignite-3/pull/5509#discussion_r2016684368 ## modules/index/src/integrationTest/java/org/apache/ignite/internal/index/ItIndexRegistrationAndRebalanceConcurrencyTest.java: ## @@ -0,0 +1,61 @@ +/* + * Licensed

Re: [PR] IGNITE-24938 Ignore attempts to create index storage in destroyed partition [ignite-3]

2025-03-27 Thread via GitHub
rpuch commented on code in PR #5509: URL: https://github.com/apache/ignite-3/pull/5509#discussion_r2016673520 ## modules/index/src/integrationTest/java/org/apache/ignite/internal/index/ItIndexRegistrationAndRebalanceConcurrencyTest.java: ## @@ -0,0 +1,61 @@ +/* + * Licensed to t