[PR] IGNITE-24188 Fix CheckpointFailBeforeWriteMarkTest (PDS 2 group) [ignite]

2025-01-20 Thread via GitHub
superminkfan opened a new pull request, #11825: URL: https://github.com/apache/ignite/pull/11825 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: ##

Re: [PR] IGNITE-24239 Create a reproducer for IGNITE-23588 [ignite-3]

2025-01-20 Thread via GitHub
ibessonov merged PR #5065: URL: https://github.com/apache/ignite-3/pull/5065 -- 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 Website: update banners [ignite-website]

2025-01-20 Thread via GitHub
hoter opened a new pull request, #190: URL: https://github.com/apache/ignite-website/pull/190 (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

Re: [PR] IGNITE-23967 Python DB API Fix fetching long result sets [ignite-3]

2025-01-20 Thread via GitHub
ptupitsyn commented on code in PR #5082: URL: https://github.com/apache/ignite-3/pull/5082#discussion_r1923148065 ## modules/platforms/python/cpp_module/utils.cpp: ## @@ -47,6 +49,12 @@ bool check_errors(ignite::diagnosable& diag) { break; case SQL_ERROR:

Re: [PR] IGNITE-24150 Ability to have different prefixes for exceptions [ignite-3]

2025-01-20 Thread via GitHub
sk0x50 commented on code in PR #4991: URL: https://github.com/apache/ignite-3/pull/4991#discussion_r1923153318 ## modules/core/src/main/java/org/apache/ignite/internal/lang/ErrorGroupHelper.java: ## @@ -0,0 +1,45 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

Re: [PR] IGNITE-24150 Ability to have different prefixes for exceptions [ignite-3]

2025-01-20 Thread via GitHub
sk0x50 commented on code in PR #4991: URL: https://github.com/apache/ignite-3/pull/4991#discussion_r1923153318 ## modules/core/src/main/java/org/apache/ignite/internal/lang/ErrorGroupHelper.java: ## @@ -0,0 +1,45 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

[PR] IGNITE-23967 Python DB API Fix fetching long result sets [ignite-3]

2025-01-20 Thread via GitHub
isapego opened a new pull request, #5082: URL: https://github.com/apache/ignite-3/pull/5082 - Added tests; - Fixed a bug that prevented the exception during fetch to be reported; - Fixed argument ordering (`timeout` and `page_size` were mixed up); - Fixed last page indicator reading.

[PR] IGNITE-24259 Improve NodeStoppingException handling [ignite-3]

2025-01-20 Thread via GitHub
rpuch opened a new pull request, #5076: URL: https://github.com/apache/ignite-3/pull/5076 https://issues.apache.org/jira/browse/IGNITE-24259 -- 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-24178 Remove redundant IdleVerify code [ignite]

2025-01-20 Thread via GitHub
nizhikov commented on PR #11801: URL: https://github.com/apache/ignite/pull/11801#issuecomment-2602188535 1. methods `PartitionKeyV2#groupName(String)`, `PartitionKey#groupName(String)` have no usage. Can be removed. 2. `PartitionKeyV2#grpName`, `PartitionKey#grpName` can become `final`.

Re: [PR] IGNITE-23918 Remove IGNITE_EXCHANGE_COMPATIBILITY_VER_1 and related code [ignite]

2025-01-20 Thread via GitHub
nizhikov closed pull request #11718: IGNITE-23918 Remove IGNITE_EXCHANGE_COMPATIBILITY_VER_1 and related code URL: https://github.com/apache/ignite/pull/11718 -- 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

Re: [PR] IGNITE-23912 Remove RMV_ANON_CLS_POS_CHANGED_SINCE [ignite]

2025-01-20 Thread via GitHub
nizhikov closed pull request #11716: IGNITE-23912 Remove RMV_ANON_CLS_POS_CHANGED_SINCE URL: https://github.com/apache/ignite/pull/11716 -- 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

Re: [PR] IGNITE-24137 Add HIGH_AVAILABILITY mode to zone suites, where valuable [ignite-3]

2025-01-20 Thread via GitHub
alievmirza commented on code in PR #5028: URL: https://github.com/apache/ignite-3/pull/5028#discussion_r1922190127 ## modules/distribution-zones/src/integrationTest/java/org/apache/ignite/internal/distributionzones/ItDistributionZonesFiltersTest.java: ## @@ -111,15 +113,20 @@ pr

Re: [PR] IGNITE-24094 Remove GridDhtPartitionDemandMessage#VERSION_SINCE [ignite]

2025-01-20 Thread via GitHub
nizhikov merged PR #11813: URL: https://github.com/apache/ignite/pull/11813 -- 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-24254 Thin client: add schema name placeholder to table ops [ignite-3]

2025-01-20 Thread via GitHub
ptupitsyn opened a new pull request, #5077: URL: https://github.com/apache/ignite-3/pull/5077 (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

Re: [PR] IGNITE-24094 Remove GridDhtPartitionDemandMessage#VERSION_SINCE [ignite]

2025-01-20 Thread via GitHub
sonarqubecloud[bot] commented on PR #11813: URL: https://github.com/apache/ignite/pull/11813#issuecomment-2602237677 ## [![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-23936 Java thin: Support broadcast partitioned [ignite-3]

2025-01-20 Thread via GitHub
Pochatkin merged PR #5068: URL: https://github.com/apache/ignite-3/pull/5068 -- 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-24194 Fix flaky testCleanStates [ignite-3]

2025-01-20 Thread via GitHub
Pochatkin merged PR #5075: URL: https://github.com/apache/ignite-3/pull/5075 -- 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-24094 Remove GridDhtPartitionDemandMessage#VERSION_SINCE [ignite]

2025-01-20 Thread via GitHub
nizhikov commented on PR #11813: URL: https://github.com/apache/ignite/pull/11813#issuecomment-2601896782 Looks like we don't need `GridDhtPartitionDemandLegacyMessage` any more. Class can be removed. Let's remove it in this PR -- This is an automated message from the Apache Git

Re: [PR] IGNITE-24094 Remove GridDhtPartitionDemandMessage#VERSION_SINCE [ignite]

2025-01-20 Thread via GitHub
sonarqubecloud[bot] commented on PR #11813: URL: https://github.com/apache/ignite/pull/11813#issuecomment-2601864253 ## [![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-24178 Remove redundant IdleVerify code [ignite]

2025-01-20 Thread via GitHub
sonarqubecloud[bot] commented on PR #11801: URL: https://github.com/apache/ignite/pull/11801#issuecomment-2601912970 ## [![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-24194 testCleanStates is flaky [ignite-3]

2025-01-20 Thread via GitHub
valepakh opened a new pull request, #5075: URL: https://github.com/apache/ignite-3/pull/5075 https://issues.apache.org/jira/browse/IGNITE-24194 -- 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

Re: [PR] IGNITE-24236 Reduce log pollution by LogicalNode [ignite-3]

2025-01-20 Thread via GitHub
sashapolo merged PR #5066: URL: https://github.com/apache/ignite-3/pull/5066 -- 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-24250 Minor refactoring in DistributionZoneManager [ignite-3]

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

Re: [PR] IGNITE-23747 Support timeouts for RO transactions [ignite-3]

2025-01-20 Thread via GitHub
rpuch merged PR #4902: URL: https://github.com/apache/ignite-3/pull/4902 -- 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-23901 Add operation time performance statistics for putAllConflict, removeAllConflict [ignite]

2025-01-20 Thread via GitHub
maksaska commented on code in PR #11793: URL: https://github.com/apache/ignite/pull/11793#discussion_r1922545542 ## modules/core/src/test/java/org/apache/ignite/internal/processors/performancestatistics/PerformanceStatisticsThinClientTest.java: ## @@ -171,6 +178,75 @@ public voi

Re: [PR] IGNITE-23901 Add operation time performance statistics for putAllConflict, removeAllConflict [ignite]

2025-01-20 Thread via GitHub
maksaska commented on code in PR #11793: URL: https://github.com/apache/ignite/pull/11793#discussion_r1922547785 ## modules/core/src/test/java/org/apache/ignite/internal/processors/performancestatistics/PerformanceStatisticsThinClientTest.java: ## @@ -171,6 +178,75 @@ public voi

Re: [PR] IGNITE-23901 Add operation time performance statistics for putAllConflict, removeAllConflict [ignite]

2025-01-20 Thread via GitHub
maksaska commented on code in PR #11793: URL: https://github.com/apache/ignite/pull/11793#discussion_r1922546435 ## modules/core/src/test/java/org/apache/ignite/internal/processors/performancestatistics/PerformanceStatisticsThinClientTest.java: ## @@ -171,6 +178,75 @@ public voi

Re: [PR] IGNITE-24256 Fixed NullPointerException during stop of TcpDiscoveryMulticastIpFinder. [ignite]

2025-01-20 Thread via GitHub
petrov-mg merged PR #11823: URL: https://github.com/apache/ignite/pull/11823 -- 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-23901 Add operation time performance statistics for putAllConflict, removeAllConflict [ignite]

2025-01-20 Thread via GitHub
maksaska commented on code in PR #11793: URL: https://github.com/apache/ignite/pull/11793#discussion_r1922545887 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/GridCacheAdapter.java: ## @@ -1969,8 +1969,9 @@ protected boolean put0(final K key, final V

Re: [PR] IGNITE-24254 Thin client: add schema name placeholder to protocol [ignite-3]

2025-01-20 Thread via GitHub
ptupitsyn commented on PR #5077: URL: https://github.com/apache/ignite-3/pull/5077#issuecomment-2602768022 @AMashenkov Everywhere else we use `int tableId`, I've double checked. We should be good. -- This is an automated message from the Apache Git Service. To respond to the message, plea

Re: [PR] IGNITE-24254 Thin client: add schema name placeholder to protocol [ignite-3]

2025-01-20 Thread via GitHub
ptupitsyn merged PR #5077: URL: https://github.com/apache/ignite-3/pull/5077 -- 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-24107 Allow to use an existing buffer in BinaryTupleBuilder [ignite-3]

2025-01-20 Thread via GitHub
xtern closed pull request #5009: IGNITE-24107 Allow to use an existing buffer in BinaryTupleBuilder URL: https://github.com/apache/ignite-3/pull/5009 -- 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

Re: [PR] IGNITE-24089 Introduce chain and link abstractions [ignite-3]

2025-01-20 Thread via GitHub
sanpwc commented on code in PR #5046: URL: https://github.com/apache/ignite-3/pull/5046#discussion_r1922736076 ## modules/partition-distribution/src/main/java/org/apache/ignite/internal/partitiondistribution/AssignmentsChain.java: ## @@ -30,15 +33,19 @@ * Contains the chain of

Re: [PR] IGNITE-24089 Introduce chain and link abstractions [ignite-3]

2025-01-20 Thread via GitHub
Cyrill commented on code in PR #5046: URL: https://github.com/apache/ignite-3/pull/5046#discussion_r1922762873 ## modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/rebalance/RebalanceRaftGroupEventsListener.java: ## @@ -493,23 +499,31 @@ priva

Re: [PR] IGNITE-24089 Introduce chain and link abstractions [ignite-3]

2025-01-20 Thread via GitHub
Cyrill commented on code in PR #5046: URL: https://github.com/apache/ignite-3/pull/5046#discussion_r1922763102 ## modules/partition-distribution/src/main/java/org/apache/ignite/internal/partitiondistribution/AssignmentsChain.java: ## @@ -30,15 +33,19 @@ * Contains the chain of

Re: [PR] IGNITE-24089 Introduce chain and link abstractions [ignite-3]

2025-01-20 Thread via GitHub
Cyrill commented on code in PR #5046: URL: https://github.com/apache/ignite-3/pull/5046#discussion_r1922765770 ## modules/partition-distribution/src/main/java/org/apache/ignite/internal/partitiondistribution/AssignmentsLinkSerializer.java: ## @@ -0,0 +1,50 @@ +/* + * Licensed to

Re: [PR] IGNITE-24089 Introduce chain and link abstractions [ignite-3]

2025-01-20 Thread via GitHub
Cyrill commented on code in PR #5046: URL: https://github.com/apache/ignite-3/pull/5046#discussion_r1922765770 ## modules/partition-distribution/src/main/java/org/apache/ignite/internal/partitiondistribution/AssignmentsLinkSerializer.java: ## @@ -0,0 +1,50 @@ +/* + * Licensed to

Re: [PR] IGNITE-24089 Introduce chain and link abstractions [ignite-3]

2025-01-20 Thread via GitHub
Cyrill commented on code in PR #5046: URL: https://github.com/apache/ignite-3/pull/5046#discussion_r1922769912 ## modules/partition-distribution/src/main/java/org/apache/ignite/internal/partitiondistribution/AssignmentsChain.java: ## @@ -30,15 +33,19 @@ * Contains the chain of

Re: [PR] IGNITE-24089 Introduce chain and link abstractions [ignite-3]

2025-01-20 Thread via GitHub
Cyrill commented on code in PR #5046: URL: https://github.com/apache/ignite-3/pull/5046#discussion_r1922768175 ## modules/transactions/src/integrationTest/java/org/apache/ignite/internal/disaster/ItDisasterRecoveryReconfigurationTest.java: ## @@ -1267,7 +1267,7 @@ void testAssig

Re: [PR] IGNITE-23085 Out of Memory preventive actions at startup [ignite]

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

[PR] IGNITE-24221 Implement new benchmarks that cover creating a distribution zone and table [ignite-3]

2025-01-20 Thread via GitHub
sk0x50 opened a new pull request, #5081: URL: https://github.com/apache/ignite-3/pull/5081 https://issues.apache.org/jira/browse/IGNITE-24221 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-24089 Introduce chain and link abstractions [ignite-3]

2025-01-20 Thread via GitHub
Cyrill commented on code in PR #5046: URL: https://github.com/apache/ignite-3/pull/5046#discussion_r1922766709 ## modules/partition-distribution/src/main/java/org/apache/ignite/internal/partitiondistribution/AssignmentsLink.java: ## @@ -0,0 +1,101 @@ +/* + * Licensed to the Apac

Re: [PR] IGNITE-24089 Introduce chain and link abstractions [ignite-3]

2025-01-20 Thread via GitHub
alievmirza commented on code in PR #5046: URL: https://github.com/apache/ignite-3/pull/5046#discussion_r1922786155 ## modules/partition-distribution/src/main/java/org/apache/ignite/internal/partitiondistribution/AssignmentsChain.java: ## @@ -30,15 +33,19 @@ * Contains the chai

Re: [PR] IGNITE-24089 Introduce chain and link abstractions [ignite-3]

2025-01-20 Thread via GitHub
alievmirza commented on code in PR #5046: URL: https://github.com/apache/ignite-3/pull/5046#discussion_r1922785947 ## modules/partition-distribution/src/main/java/org/apache/ignite/internal/partitiondistribution/AssignmentsChain.java: ## @@ -30,15 +33,19 @@ * Contains the chai

Re: [PR] IGNITE-24178 Remove redundant IdleVerify code [ignite]

2025-01-20 Thread via GitHub
maksaska commented on PR #11801: URL: https://github.com/apache/ignite/pull/11801#issuecomment-2602601539 * PartitionKeyV2 is removed * PartitionKey is fixed -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] IGNITE-24089 Introduce chain and link abstractions [ignite-3]

2025-01-20 Thread via GitHub
Cyrill commented on code in PR #5046: URL: https://github.com/apache/ignite-3/pull/5046#discussion_r1922803506 ## modules/partition-distribution/src/main/java/org/apache/ignite/internal/partitiondistribution/AssignmentsChain.java: ## @@ -64,31 +71,70 @@ public AssignmentsChain r

Re: [PR] IGNITE-24089 Introduce chain and link abstractions [ignite-3]

2025-01-20 Thread via GitHub
Cyrill commented on code in PR #5046: URL: https://github.com/apache/ignite-3/pull/5046#discussion_r1922803376 ## modules/partition-distribution/src/main/java/org/apache/ignite/internal/partitiondistribution/AssignmentsChain.java: ## @@ -64,31 +71,70 @@ public AssignmentsChain r

[PR] Ignite-24212 : non persistent sql schema v2 [ignite]

2025-01-20 Thread via GitHub
Vladsz83 opened a new pull request, #11824: URL: https://github.com/apache/ignite/pull/11824 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

[PR] IGNITE-24264 Remove Google & Yandex Analytics from the Ignite Website [ignite-website]

2025-01-20 Thread via GitHub
niallkp opened a new pull request, #189: URL: https://github.com/apache/ignite-website/pull/189 See https://issues.apache.org/jira/browse/IGNITE-24264 -- 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-24030: Schema commands DDL handler. [ignite-3]

2025-01-20 Thread via GitHub
zstan merged PR #5021: URL: https://github.com/apache/ignite-3/pull/5021 -- 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-24178 Remove redundant IdleVerify code [ignite]

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

[PR] Bump org.assertj:assertj-core from 3.27.2 to 3.27.3 [ignite-3]

2025-01-20 Thread via GitHub
dependabot[bot] opened a new pull request, #5079: URL: https://github.com/apache/ignite-3/pull/5079 Bumps [org.assertj:assertj-core](https://github.com/assertj/assertj) from 3.27.2 to 3.27.3. Release notes Sourced from https://github.com/assertj/assertj/releases";>org.assertj:asser

[PR] IGNITE-23489 Add rest endpoint to retrieve active transactions and running queries [ignite-3]

2025-01-20 Thread via GitHub
nevdmitry opened a new pull request, #5080: URL: https://github.com/apache/ignite-3/pull/5080 Introduce new rest endpoints GET /management/v1/transactions/ GET /management/v1/transactions/{tx_id} DELETE /management/v1/transactions/{tx_id} GET /management/v1/sql/queries/

[PR] Bump net.bytebuddy:byte-buddy from 1.15.11 to 1.16.1 [ignite-3]

2025-01-20 Thread via GitHub
dependabot[bot] opened a new pull request, #5078: URL: https://github.com/apache/ignite-3/pull/5078 Bumps [net.bytebuddy:byte-buddy](https://github.com/raphw/byte-buddy) from 1.15.11 to 1.16.1. Release notes Sourced from https://github.com/raphw/byte-buddy/releases";>net.bytebuddy:

Re: [PR] IGNITE-24254 Thin client: add schema name placeholder to protocol [ignite-3]

2025-01-20 Thread via GitHub
AMashenkov commented on PR #5077: URL: https://github.com/apache/ignite-3/pull/5077#issuecomment-2602740625 @ptupitsyn Do we need a similar fix for Compute API or anything else? I've mention only the classes, where I saw failures in my branch (these ones you've changed), but I could miss

Re: [PR] IGNITE-24135 BPlusTree reuses lastRow from closure to fill buffer [ignite]

2025-01-20 Thread via GitHub
nizhikov commented on code in PR #11816: URL: https://github.com/apache/ignite/pull/11816#discussion_r1922713778 ## modules/core/src/main/java/org/apache/ignite/internal/cache/query/index/sorted/inline/InlineTreeFilterClosure.java: ## @@ -56,12 +60,20 @@ public InlineTreeFilterC

Re: [PR] IGNITE-24135 BPlusTree reuses lastRow from closure to fill buffer [ignite]

2025-01-20 Thread via GitHub
nizhikov commented on code in PR #11816: URL: https://github.com/apache/ignite/pull/11816#discussion_r1922712760 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/tree/BPlusTree.java: ## @@ -5724,8 +5724,16 @@ private final class ForwardCursor