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

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

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

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

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

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

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

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

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

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

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

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

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

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

Re: [PR] IGNITE-24927 Fix table storages destruction for per-zone partitions [ignite-3]

2025-03-26 Thread via GitHub
rpuch merged PR #5499: URL: https://github.com/apache/ignite-3/pull/5499 -- 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

[PR] IGNITE-15803 Remove instanceof BinaryMarshaller [ignite]

2025-03-26 Thread via GitHub
nizhikov opened a new pull request, #11956: URL: https://github.com/apache/ignite/pull/11956 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

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

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

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

2025-03-26 Thread via GitHub
nizhikov commented on code in PR #11939: URL: https://github.com/apache/ignite/pull/11939#discussion_r2013892194 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -0,0 +1,459 @@ +/* + * 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-26 Thread via GitHub
xtern commented on code in PR #5493: URL: https://github.com/apache/ignite-3/pull/5493#discussion_r2014230025 ## modules/table/src/integrationTest/java/org/apache/ignite/internal/table/ItKeyValueViewSimpleSchemaApiTest.java: ## @@ -60,28 +60,38 @@ public class ItKeyValueViewSimp

Re: [PR] IGNITE-23266 Add metrics of waiting to complete page replacement on checkpoint [ignite-3]

2025-03-26 Thread via GitHub
tkalkirill merged PR #5500: URL: https://github.com/apache/ignite-3/pull/5500 -- 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-24863 Rebuild ignite 2.15 doc [ignite-website]

2025-03-26 Thread via GitHub
ptupitsyn merged PR #218: URL: https://github.com/apache/ignite-website/pull/218 -- 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-24795 Sql. Add PROJECT_CORRELATE rule to HEP push down list [ignite-3]

2025-03-26 Thread via GitHub
zstan commented on code in PR #5437: URL: https://github.com/apache/ignite-3/pull/5437#discussion_r2013585567 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/rule/logical/IgniteProjectCorrelateTransposeRule.java: ## @@ -0,0 +1,259 @@ +/* + * Licensed to

Re: [PR] IGNITE-24871 Rebuild ignite 2.11.1 doc [ignite-website]

2025-03-26 Thread via GitHub
ptupitsyn merged PR #214: URL: https://github.com/apache/ignite-website/pull/214 -- 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-24870 Rebuild ignite 2.9.1 docs [ignite-website]

2025-03-26 Thread via GitHub
ptupitsyn merged PR #211: URL: https://github.com/apache/ignite-website/pull/211 -- 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-24862 Rebuild ignite 2.16 doc [ignite-website]

2025-03-26 Thread via GitHub
ptupitsyn merged PR #219: URL: https://github.com/apache/ignite-website/pull/219 -- 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-24782 Initial commit for ignite-commons [ignite]

2025-03-26 Thread via GitHub
nizhikov merged PR #11942: URL: https://github.com/apache/ignite/pull/11942 -- 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] Bump com.google.guava:guava from 33.4.5-jre to 33.4.6-jre [ignite-3]

2025-03-26 Thread via GitHub
dependabot[bot] opened a new pull request, #5503: URL: https://github.com/apache/ignite-3/pull/5503 Bumps [com.google.guava:guava](https://github.com/google/guava) from 33.4.5-jre to 33.4.6-jre. Release notes Sourced from https://github.com/google/guava/releases";>com.google.guava:

[PR] Bump com.netflix.nebula.ospackage from 11.11.1 to 11.11.2 [ignite-3]

2025-03-26 Thread via GitHub
dependabot[bot] opened a new pull request, #5504: URL: https://github.com/apache/ignite-3/pull/5504 Bumps com.netflix.nebula.ospackage from 11.11.1 to 11.11.2. [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=com

[PR] IGNITE-24860 Rebuild Ignite 3 doc [ignite-website]

2025-03-26 Thread via GitHub
IgGusev opened a new pull request, #227: URL: https://github.com/apache/ignite-website/pull/227 (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-26 Thread via GitHub
nizhikov commented on code in PR #11939: URL: https://github.com/apache/ignite/pull/11939#discussion_r2013901049 ## modules/compatibility/src/test/java/org/apache/ignite/compatibility/persistence/SnapshotCompatibilityTest.java: ## @@ -0,0 +1,459 @@ +/* + * Licensed to the Apache

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

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

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

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

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

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

[PR] IGNITE-24535 Compatibility test for NodeFileTree [ignite]

2025-03-26 Thread via GitHub
vladnovoren opened a new pull request, #11969: URL: https://github.com/apache/ignite/pull/11969 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: ###

(ignite-website) branch master updated (e5ffabcc1b -> 5ab7682b1f)

2025-03-26 Thread ptupitsyn
This is an automated email from the ASF dual-hosted git repository. ptupitsyn pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/ignite-website.git from e5ffabcc1b IGNITE-24871 Rebuild ignite 2.11.1 doc (#214) add 5ab7682b1f IGNITE-24867 Rebuild 2.11 doc

Re: [PR] IGNITE-24860 Rebuild ignite 3 doc [ignite-website]

2025-03-26 Thread via GitHub
IgGusev closed pull request #221: IGNITE-24860 Rebuild ignite 3 doc URL: https://github.com/apache/ignite-website/pull/221 -- 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 un

[PR] IGNITE-24930 Catalog. Write protocol version into metastore on startup [ignite-3]

2025-03-26 Thread via GitHub
lowka opened a new pull request, #5502: URL: https://github.com/apache/ignite-3/pull/5502 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-24910 Fix flaky killComputeJobFromRemote test [ignite-3]

2025-03-26 Thread via GitHub
PakhomovAlexander merged PR #5495: URL: https://github.com/apache/ignite-3/pull/5495 -- 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

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

2025-03-26 Thread via GitHub
sonarqubecloud[bot] commented on PR #11967: URL: https://github.com/apache/ignite/pull/11967#issuecomment-2754979147 ## [![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-24867 Rebuild 2.11 doc [ignite-website]

2025-03-26 Thread via GitHub
ptupitsyn merged PR #213: URL: https://github.com/apache/ignite-website/pull/213 -- 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-24866 Rebuild ignite 2.12 doc [ignite-website]

2025-03-26 Thread via GitHub
ptupitsyn merged PR #217: URL: https://github.com/apache/ignite-website/pull/217 -- 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-24860 Rebuilding Ignite 3 doc [ignite-website]

2025-03-26 Thread via GitHub
IgGusev closed pull request #226: IGNITE-24860 Rebuilding Ignite 3 doc URL: https://github.com/apache/ignite-website/pull/226 -- 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

[PR] IGNITE-24860 Rebuilding Ignite 3 doc [ignite-website]

2025-03-26 Thread via GitHub
IgGusev opened a new pull request, #226: URL: https://github.com/apache/ignite-website/pull/226 (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-24788 Configurable coalescing for RAFT heartbeat [ignite-3]

2025-03-26 Thread via GitHub
vldpyatkov commented on code in PR #5477: URL: https://github.com/apache/ignite-3/pull/5477#discussion_r2014423682 ## modules/raft/src/main/java/org/apache/ignite/raft/jraft/core/NodeImpl.java: ## @@ -1327,29 +1313,31 @@ private void initPools(final NodeOptions opts) {

(ignite-website) branch master updated (d54e941440 -> 2e97a46823)

2025-03-26 Thread ptupitsyn
This is an automated email from the ASF dual-hosted git repository. ptupitsyn pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/ignite-website.git from d54e941440 IGNITE-24861 Rebuild ignite 2.17 doc (#220) add 2e97a46823 IGNITE-24862 Rebuild ignite 2.16

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

2025-03-26 Thread via GitHub
xtern commented on code in PR #5490: URL: https://github.com/apache/ignite-3/pull/5490#discussion_r2014374125 ## modules/sql-engine/src/test/java/org/apache/ignite/internal/sql/engine/sql/SqlDdlParserTest.java: ## @@ -767,15 +770,36 @@ public void dropIndexIfExists() { }

Re: [PR] IGNITE-24868 Rebuild ignite 2.10 doc [ignite-website]

2025-03-26 Thread via GitHub
ptupitsyn merged PR #212: URL: https://github.com/apache/ignite-website/pull/212 -- 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-24865 Rebuild ignite 2.13 doc [ignite-website]

2025-03-26 Thread via GitHub
ptupitsyn merged PR #216: URL: https://github.com/apache/ignite-website/pull/216 -- 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-24861 Rebuild ignite 2.17 doc [ignite-website]

2025-03-26 Thread via GitHub
ptupitsyn merged PR #220: URL: https://github.com/apache/ignite-website/pull/220 -- 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-23919 Remove DurableBackgroundCleanupIndexTreeTask and related code [ignite]

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

[I] Does Apache Ignite 3 successfully working with .NET 8? [ignite]

2025-03-26 Thread via GitHub
Maximys opened a new issue, #11958: URL: https://github.com/apache/ignite/issues/11958 I tried to figure out how to use Apache Ignite with a .NET application. I created a solution for this [here](https://github.com/Maximys/reports/tree/dev/HTC/Redis/src/RedisForDummies). But for some reaso

Re: [PR] IGNITE-24788 Configurable coalescing for RAFT heartbeat [ignite-3]

2025-03-26 Thread via GitHub
alievmirza commented on code in PR #5477: URL: https://github.com/apache/ignite-3/pull/5477#discussion_r2014085610 ## modules/raft/src/integrationTest/java/org/apache/ignite/internal/raft/ItRaftGroupServiceTest.java: ## @@ -206,10 +216,73 @@ public void testChangePeersAndLearner

Re: [PR] IGNITE-23167 Cleaning up redundant warning log output in GridSqlQuery… [ignite]

2025-03-26 Thread via GitHub
liyuj commented on code in PR #11521: URL: https://github.com/apache/ignite/pull/11521#discussion_r2015427714 ## modules/indexing/src/main/java/org/apache/ignite/internal/processors/query/h2/sql/GridSqlQueryParser.java: ## @@ -826,11 +826,6 @@ private GridSqlMerge parseMerge(Mer

Re: [PR] IGNITE-23167 Cleaning up redundant warning log output in GridSqlQuery… [ignite]

2025-03-26 Thread via GitHub
liyuj closed pull request #11521: IGNITE-23167 Cleaning up redundant warning log output in GridSqlQuery… URL: https://github.com/apache/ignite/pull/11521 -- 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

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

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

Re: [PR] IGNITE-24795 Sql. Add PROJECT_CORRELATE rule to HEP push down list [ignite-3]

2025-03-26 Thread via GitHub
ygerzhedovich commented on code in PR #5437: URL: https://github.com/apache/ignite-3/pull/5437#discussion_r2013876772 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/rule/logical/IgniteProjectCorrelateTransposeRule.java: ## @@ -0,0 +1,259 @@ +/* + * Lic

[PR] IGNITE-24932 Fix hanging of module runner tests [ignite-3]

2025-03-26 Thread via GitHub
sk0x50 opened a new pull request, #5501: URL: https://github.com/apache/ignite-3/pull/5501 https://issues.apache.org/jira/browse/IGNITE-24932 Thank you for submitting the pull request. To streamline the review process of the patch and ensure better code quality we ask both an

Re: [PR] IGNITE-24716 Sql. MergeJoin is failed with IndexOutOfBoundsException [ignite-3]

2025-03-26 Thread via GitHub
AMashenkov merged PR #5454: URL: https://github.com/apache/ignite-3/pull/5454 -- 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-24910 Fix flaky killComputeJobFromRemote test [ignite-3]

2025-03-26 Thread via GitHub
valepakh commented on code in PR #5495: URL: https://github.com/apache/ignite-3/pull/5495#discussion_r2013679930 ## modules/compute/src/main/java/org/apache/ignite/internal/compute/ExecutionManager.java: ## @@ -123,7 +123,7 @@ public CompletableFuture> localStatesAsync() {

Re: [PR] IGNITE-15803 Remove instanceOf BinaryMarshaller [ignite]

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

Re: [PR] IGNITE-24702 Fix of CompactedException on the end of rebalancing [ignite-3]

2025-03-26 Thread via GitHub
alievmirza commented on code in PR #5468: URL: https://github.com/apache/ignite-3/pull/5468#discussion_r2013982988 ## modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/DataNodesHistory.java: ## @@ -88,6 +88,11 @@ public DataNodesHistoryEntry

Re: [PR] IGNITE-23266 Add metrics of waiting to complete page replacement on checkpoint [ignite-3]

2025-03-26 Thread via GitHub
tkalkirill commented on code in PR #5500: URL: https://github.com/apache/ignite-3/pull/5500#discussion_r2013994916 ## modules/page-memory/src/test/java/org/apache/ignite/internal/pagememory/persistence/checkpoint/CheckpointMetricsTrackerTest.java: ## @@ -171,6 +171,19 @@ void te

Re: [PR] IGNITE-24708 Sql. Improve CREATE ZONE syntax [ignite-3]

2025-03-26 Thread via GitHub
korlov42 commented on code in PR #5451: URL: https://github.com/apache/ignite-3/pull/5451#discussion_r201300 ## modules/sql-engine/src/main/codegen/includes/parserImpls.ftl: ## @@ -527,16 +527,176 @@ SqlCreate SqlCreateZone(Span s, boolean replace) : final boolean i

Re: [PR] IGNITE-26266 Add metrics of waiting to complete page replacement on checkpoint [ignite-3]

2025-03-26 Thread via GitHub
ivanzlenko commented on code in PR #5500: URL: https://github.com/apache/ignite-3/pull/5500#discussion_r2013967158 ## modules/page-memory/src/test/java/org/apache/ignite/internal/pagememory/persistence/checkpoint/CheckpointMetricsTrackerTest.java: ## @@ -171,6 +171,19 @@ void te

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

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

Re: [PR] IGNITE-24702 Fix of CompactedException on the end of rebalancing [ignite-3]

2025-03-26 Thread via GitHub
alievmirza commented on code in PR #5468: URL: https://github.com/apache/ignite-3/pull/5468#discussion_r2014021474 ## modules/catalog/src/main/java/org/apache/ignite/internal/catalog/descriptors/CatalogSchemaDescriptorSerializers.java: ## @@ -57,13 +61,16 @@ public CatalogSchema

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

2025-03-26 Thread via GitHub
zstan commented on code in PR #5490: URL: https://github.com/apache/ignite-3/pull/5490#discussion_r2013543299 ## modules/sql-engine/src/test/java/org/apache/ignite/internal/sql/engine/sql/SqlDdlParserTest.java: ## @@ -767,15 +770,36 @@ public void dropIndexIfExists() { }

[PR] IGNITE-24927 Fix table storages destruction for per-zone partitions [ignite-3]

2025-03-26 Thread via GitHub
rpuch opened a new pull request, #5499: URL: https://github.com/apache/ignite-3/pull/5499 https://issues.apache.org/jira/browse/IGNITE-24927 -- 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

[PR] IGNITE-26266 Add metrics of waiting to complete page replacement on checkpoint [ignite-3]

2025-03-26 Thread via GitHub
tkalkirill opened a new pull request, #5500: URL: https://github.com/apache/ignite-3/pull/5500 https://issues.apache.org/jira/browse/IGNITE-23266 -- 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

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

2025-03-26 Thread via GitHub
xtern commented on code in PR #5490: URL: https://github.com/apache/ignite-3/pull/5490#discussion_r2013590054 ## modules/sql-engine/src/test/java/org/apache/ignite/internal/sql/engine/sql/SqlDdlParserTest.java: ## @@ -767,15 +770,36 @@ public void dropIndexIfExists() { }

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

2025-03-26 Thread via GitHub
xtern commented on code in PR #5490: URL: https://github.com/apache/ignite-3/pull/5490#discussion_r2013595620 ## modules/sql-engine/src/test/java/org/apache/ignite/internal/sql/engine/sql/SqlDdlParserTest.java: ## @@ -767,15 +770,36 @@ public void dropIndexIfExists() { }

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

2025-03-26 Thread via GitHub
xtern commented on code in PR #5490: URL: https://github.com/apache/ignite-3/pull/5490#discussion_r2013590054 ## modules/sql-engine/src/test/java/org/apache/ignite/internal/sql/engine/sql/SqlDdlParserTest.java: ## @@ -767,15 +770,36 @@ public void dropIndexIfExists() { }

Re: [PR] IGNITE-24227 Sql. Improve type checking in LIMIT / OFFSET clauses [ignite-3]

2025-03-26 Thread via GitHub
korlov42 commented on code in PR #5479: URL: https://github.com/apache/ignite-3/pull/5479#discussion_r2013605481 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/prepare/IgniteSqlValidator.java: ## @@ -700,34 +700,48 @@ private void checkIntegerLimit(@Nu

Re: [I] Ignite Native query not comparing when Colum value has '-' in value name [ignite]

2025-03-26 Thread via GitHub
kagrawal-tibco commented on issue #11963: URL: https://github.com/apache/ignite/issues/11963#issuecomment-2753407742 Thank you Pavel for a quick response I tested with primary key column . I will try to attach the project. -- This is an automated message from the Apache Git Service. To

Re: [PR] IGNITE-24919 Make non-blocking get a compaction revision for the MetaStorageManager [ignite-3]

2025-03-26 Thread via GitHub
tkalkirill merged PR #5497: URL: https://github.com/apache/ignite-3/pull/5497 -- 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-24535 Compatibility test for Ignite snapshot [ignite]

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

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

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

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

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

Re: [PR] IGNITE-24860 Rebuild Ignite 3 doc [ignite-website]

2025-03-26 Thread via GitHub
ptupitsyn merged PR #227: URL: https://github.com/apache/ignite-website/pull/227 -- 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-24227 Sql. Improve type checking in LIMIT / OFFSET clauses [ignite-3]

2025-03-26 Thread via GitHub
zstan commented on code in PR #5479: URL: https://github.com/apache/ignite-3/pull/5479#discussion_r2015670827 ## modules/sql-engine/src/integrationTest/sql/group1/order/test_limit.test: ## @@ -17,6 +17,18 @@ SELECT a FROM test ORDER BY a LIMIT 1 11 +# decimal limit +que

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

2025-03-26 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

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

2025-03-26 Thread via GitHub
rpuch opened a new pull request, #5506: URL: https://github.com/apache/ignite-3/pull/5506 https://issues.apache.org/jira/browse/IGNITE-24935 -- 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-24227 Sql. Improve type checking in LIMIT / OFFSET clauses [ignite-3]

2025-03-26 Thread via GitHub
zstan commented on code in PR #5479: URL: https://github.com/apache/ignite-3/pull/5479#discussion_r2015661860 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/LogicalRelImplementor.java: ## @@ -1158,4 +1144,20 @@ private static boolean canFuseProjec

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

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

Re: [PR] IGNITE-24227 Sql. Improve type checking in LIMIT / OFFSET clauses [ignite-3]

2025-03-26 Thread via GitHub
zstan commented on code in PR #5479: URL: https://github.com/apache/ignite-3/pull/5479#discussion_r2015671852 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/prepare/IgniteSqlValidator.java: ## @@ -700,34 +700,48 @@ private void checkIntegerLimit(@Nulla