Re: [I] Apache Ignite 2.15.0 startup failure using OpenJDK 17 version [ignite]

2024-06-11 Thread via GitHub
Litvinovis commented on issue #10747: URL: https://github.com/apache/ignite/issues/10747#issuecomment-2161414393 So i faced the same problem when i start Ignite on windows 10, this solved in my case: 1) switch to ignite 2.16 2) added VM option:

[PR] Bump netty from 4.1.110.Final to 4.1.111.Final [ignite-3]

2024-06-11 Thread via GitHub
dependabot[bot] opened a new pull request, #3906: URL: https://github.com/apache/ignite-3/pull/3906 Bumps `netty` from 4.1.110.Final to 4.1.111.Final. Updates `io.netty:netty-common` from 4.1.110.Final to 4.1.111.Final Commits

[PR] Bump com.github.spotbugs from 6.0.15 to 6.0.16 [ignite-3]

2024-06-11 Thread via GitHub
dependabot[bot] opened a new pull request, #3905: URL: https://github.com/apache/ignite-3/pull/3905 Bumps com.github.spotbugs from 6.0.15 to 6.0.16. [![Dependabot compatibility

[PR] IGNITE-18372 metrics for k2i consumer [ignite-extensions]

2024-06-11 Thread via GitHub
maksaska opened a new pull request, #277: URL: https://github.com/apache/ignite-extensions/pull/277 ### Metrics for CDC with Kafka **CDC with Kafka** doesn't have metrics for 'Kafka To Ignite' consumer. The improvement in this pull request contains the neccessary updates to fix this

[PR] IGNITE-21973: Extend test coverage for SQL F201(CAST function) [ignite-3]

2024-06-11 Thread via GitHub
lowka opened a new pull request, #3904: URL: https://github.com/apache/ignite-3/pull/3904 Adds sqllogic tests for CAST function. https://issues.apache.org/jira/browse/IGNITE-21973 --- Thank you for submitting the pull request. To streamline the review process of

[PR] IGNITE-22396 Update xstream to 1.4.20 [ignite]

2024-06-11 Thread via GitHub
lordgarrish opened a new pull request, #11388: URL: https://github.com/apache/ignite/pull/11388 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-22302 Add resultSubscriber support to Data Streamer [ignite-3]

2024-06-11 Thread via GitHub
ptupitsyn merged PR #3900: URL: https://github.com/apache/ignite-3/pull/3900 -- 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:

[PR] IGNITE-22052: Fix authentication tests [ignite-3]

2024-06-11 Thread via GitHub
isapego opened a new pull request, #3903: URL: https://github.com/apache/ignite-3/pull/3903 https://issues.apache.org/jira/browse/IGNITE-22052 -- 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-22443 Fix sporadic configuration tests failures [ignite-3]

2024-06-11 Thread via GitHub
ibessonov merged PR #3895: URL: https://github.com/apache/ignite-3/pull/3895 -- 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-22449 Cache View documentation update [ignite]

2024-06-11 Thread via GitHub
alex-plekhanov merged PR #11385: URL: https://github.com/apache/ignite/pull/11385 -- 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:

[PR] IGNITE-22464 Java thin client: Add events for client start and stop [ignite]

2024-06-11 Thread via GitHub
alex-plekhanov opened a new pull request, #11387: URL: https://github.com/apache/ignite/pull/11387 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-22399 Improve handling of small network messages on send [ignite-3]

2024-06-11 Thread via GitHub
tkalkirill commented on code in PR #3865: URL: https://github.com/apache/ignite-3/pull/3865#discussion_r1634814772 ## modules/network/src/main/java/org/apache/ignite/internal/network/netty/PipelineUtils.java: ## @@ -57,6 +58,9 @@ public static void setup(ChannelPipeline

Re: [PR] IGNITE-21243 Bump calcite version to 1.37 [ignite-3]

2024-06-11 Thread via GitHub
ygerzhedovich merged PR #3878: URL: https://github.com/apache/ignite-3/pull/3878 -- 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-22438 Enable ItFileTransferTest [ignite-3]

2024-06-11 Thread via GitHub
Pochatkin merged PR #3901: URL: https://github.com/apache/ignite-3/pull/3901 -- 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-22204: Sql. Set operation. Incorrect query transformation for a query with limit / offset and sort [ignite-3]

2024-06-11 Thread via GitHub
korlov42 merged PR #3857: URL: https://github.com/apache/ignite-3/pull/3857 -- 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-21957 Sql. Extend test coverage for SQL E021-10 (Character string types. Implicit casting among the character string types) [ignite-3]

2024-06-11 Thread via GitHub
AMashenkov merged PR #3896: URL: https://github.com/apache/ignite-3/pull/3896 -- 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-22440 Add Maven command for building .zip files of Ignite extensions [ignite-extensions]

2024-06-11 Thread via GitHub
NSAmelchev merged PR #274: URL: https://github.com/apache/ignite-extensions/pull/274 -- 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: [I] Unable to connect to ignite with persistence enabled [ignite]

2024-06-11 Thread via GitHub
siddharth-u commented on issue #11386: URL: https://github.com/apache/ignite/issues/11386#issuecomment-2160074235 To summarise: Ignite version < 2.16 - works with persistence enabled Ignite version 2.16 - fails with above error for persistence enabled -- This is an automated

[I] Unable to connect to ignite with persistence enabled [ignite]

2024-06-11 Thread via GitHub
siddharth-u opened a new issue, #11386: URL: https://github.com/apache/ignite/issues/11386 Hi I am trying to run ignite in a cluster on kubernetes. My service tries to connect to the running ignite pod. But with ignite 2.16, the connection fails when persistence is enabled. The same

Re: [PR] IGNITE-22319 Fixed node crashing if a snapshot restore cancelled due to network issues [ignite]

2024-06-11 Thread via GitHub
NSAmelchev commented on code in PR #11361: URL: https://github.com/apache/ignite/pull/11361#discussion_r1634399429 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/SnapshotRestoreProcess.java: ## @@ -1200,8 +1200,7 @@ private

Re: [PR] IGNITE-20650 Fix case-sensitive field names resolution [ignite-3]

2024-06-11 Thread via GitHub
AMashenkov merged PR #3850: URL: https://github.com/apache/ignite-3/pull/3850 -- 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-21966: Extend test coverage for SQL E091-01(Set functions. AVG) [ignite-3]

2024-06-11 Thread via GitHub
ygerzhedovich merged PR #3898: URL: https://github.com/apache/ignite-3/pull/3898 -- 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:

[PR] IGNITE-19910 Add consumer poll timeout to kafka-cdc [ignite-extensions]

2024-06-10 Thread via GitHub
ivandasch opened a new pull request, #276: URL: https://github.com/apache/ignite-extensions/pull/276 (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

Re: [PR] IGNITE-22450 Fix ML examples tests [ignite-extensions]

2024-06-10 Thread via GitHub
ivandasch merged PR #275: URL: https://github.com/apache/ignite-extensions/pull/275 -- 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:

[PR] IGNITE-22449 Cache View documentation update [ignite]

2024-06-10 Thread via GitHub
maksaska opened a new pull request, #11385: URL: https://github.com/apache/ignite/pull/11385 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-22449 cache view documentation update [ignite]

2024-06-10 Thread via GitHub
maksaska closed pull request #11384: IGNITE-22449 cache view documentation update URL: https://github.com/apache/ignite/pull/11384 -- 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

[PR] Bump io.dropwizard.metrics:metrics-core from 4.2.25 to 4.2.26 [ignite-3]

2024-06-10 Thread via GitHub
dependabot[bot] opened a new pull request, #3902: URL: https://github.com/apache/ignite-3/pull/3902 Bumps [io.dropwizard.metrics:metrics-core](https://github.com/dropwizard/metrics) from 4.2.25 to 4.2.26. Release notes Sourced from

[PR] IGNITE-22450 fix test [ignite-extensions]

2024-06-10 Thread via GitHub
nao-it opened a new pull request, #275: URL: https://github.com/apache/ignite-extensions/pull/275 (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

[PR] IGNITE-22449 cache view documentation update [ignite]

2024-06-10 Thread via GitHub
maksaska opened a new pull request, #11384: URL: https://github.com/apache/ignite/pull/11384 Cache View documentation update with new column `HAS_EXPIRING_ENTRIES` description. Thank you for submitting the pull request to the Apache Ignite. In order to streamline the review of

Re: [PR] IGNITE-22449 cache view documentation update [ignite]

2024-06-10 Thread via GitHub
maksaska closed pull request #11383: IGNITE-22449 cache view documentation update URL: https://github.com/apache/ignite/pull/11383 -- 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-22189 cache view documentation update [ignite]

2024-06-10 Thread via GitHub
maksaska closed pull request #11383: IGNITE-22189 cache view documentation update URL: https://github.com/apache/ignite/pull/11383 -- 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

[PR] IGNITE-22189 cache view documentation update [ignite]

2024-06-10 Thread via GitHub
maksaska opened a new pull request, #11383: URL: https://github.com/apache/ignite/pull/11383 Cache View documentation is updated with new column `HAS_EXPIRING_ENTRIES` description. Thank you for submitting the pull request to the Apache Ignite. In order to streamline the

[PR] Realise/161 [ignite]

2024-06-10 Thread via GitHub
nao-it opened a new pull request, #11382: URL: https://github.com/apache/ignite/pull/11382 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

Re: [PR] IGNITE-22319 Fixed node crashing if a snapshot restore cancelled due to network issues [ignite]

2024-06-10 Thread via GitHub
Vladsz83 commented on code in PR #11361: URL: https://github.com/apache/ignite/pull/11361#discussion_r1633356267 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/IgniteSnapshotManager.java: ## @@ -3941,7 +3927,7 @@ else if (msg

Re: [PR] IGNITE-22319 Fixed node crashing if a snapshot restore cancelled due to network issues [ignite]

2024-06-10 Thread via GitHub
Vladsz83 commented on code in PR #11361: URL: https://github.com/apache/ignite/pull/11361#discussion_r1633356267 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/IgniteSnapshotManager.java: ## @@ -3941,7 +3927,7 @@ else if (msg

Re: [PR] IGNITE-22319 Fixed node crashing if a snapshot restore cancelled due to network issues [ignite]

2024-06-10 Thread via GitHub
Vladsz83 commented on code in PR #11361: URL: https://github.com/apache/ignite/pull/11361#discussion_r1633356267 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/IgniteSnapshotManager.java: ## @@ -3941,7 +3927,7 @@ else if (msg

Re: [PR] Test/161 [ignite]

2024-06-10 Thread via GitHub
nao-it closed pull request #11381: Test/161 URL: https://github.com/apache/ignite/pull/11381 -- 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:

[PR] Test/161 [ignite]

2024-06-10 Thread via GitHub
nao-it opened a new pull request, #11381: URL: https://github.com/apache/ignite/pull/11381 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

Re: [PR] IGNITE-22439 Add tests for duplicate keys in Calcite [ignite]

2024-06-10 Thread via GitHub
alex-plekhanov merged PR #11380: URL: https://github.com/apache/ignite/pull/11380 -- 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-22189 expiry policy info in CacheView and CacheGroupView [ignite]

2024-06-10 Thread via GitHub
asfgit closed pull request #11357: IGNITE-22189 expiry policy info in CacheView and CacheGroupView URL: https://github.com/apache/ignite/pull/11357 -- 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-22319 Fixed node crashing if a snapshot restore cancelled due to network issues [ignite]

2024-06-10 Thread via GitHub
Vladsz83 commented on code in PR #11361: URL: https://github.com/apache/ignite/pull/11361#discussion_r1633241269 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/IgniteSnapshotManager.java: ## @@ -3778,54 +3777,41 @@ private

Re: [PR] IGNITE-22319 Fixed node crashing if a snapshot restore cancelled due to network issues [ignite]

2024-06-10 Thread via GitHub
Vladsz83 commented on code in PR #11361: URL: https://github.com/apache/ignite/pull/11361#discussion_r1633241269 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/IgniteSnapshotManager.java: ## @@ -3778,54 +3777,41 @@ private

[PR] IGNITE-22438 Enable ItFileTransferTest [ignite-3]

2024-06-10 Thread via GitHub
valepakh opened a new pull request, #3901: URL: https://github.com/apache/ignite-3/pull/3901 https://issues.apache.org/jira/browse/IGNITE-22438 -- 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-22319 Fixed node crashing if a snapshot restore cancelled due to network issues [ignite]

2024-06-10 Thread via GitHub
Vladsz83 commented on code in PR #11361: URL: https://github.com/apache/ignite/pull/11361#discussion_r1633203984 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/SnapshotRestoreProcess.java: ## @@ -1200,8 +1200,7 @@ private

Re: [PR] IGNITE-22319 Fixed node crashing if a snapshot restore cancelled due to network issues [ignite]

2024-06-10 Thread via GitHub
NSAmelchev commented on code in PR #11361: URL: https://github.com/apache/ignite/pull/11361#discussion_r1633193314 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/IgniteSnapshotManager.java: ## @@ -3785,47 +3784,32 @@ public

Re: [PR] IGNITE-22319 Fixed node crashing if a snapshot restore cancelled due to network issues [ignite]

2024-06-10 Thread via GitHub
Vladsz83 commented on code in PR #11361: URL: https://github.com/apache/ignite/pull/11361#discussion_r1633187029 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/IgniteSnapshotManager.java: ## @@ -3785,47 +3784,32 @@ public

Re: [PR] IGNITE-22319 Fixed node crashing if a snapshot restore cancelled due to network issues [ignite]

2024-06-10 Thread via GitHub
NSAmelchev commented on code in PR #11361: URL: https://github.com/apache/ignite/pull/11361#discussion_r1633176147 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/IgniteSnapshotManager.java: ## @@ -3895,20 +3879,22 @@ private Set

Re: [PR] IGNITE-22423 Don't use kill -9 for ignite3db [ignite-3]

2024-06-10 Thread via GitHub
YuryYudin commented on code in PR #3894: URL: https://github.com/apache/ignite-3/pull/3894#discussion_r1633170871 ## packaging/db/zip/linux/ignite3db: ## @@ -59,7 +59,20 @@ start() { } stop() { Review Comment: What about printing notification about the fact that node

Re: [PR] IGNITE-22393 Fix incorrect BinaryMode in FieldAccessor for column data [ignite-3]

2024-06-10 Thread via GitHub
AMashenkov merged PR #3892: URL: https://github.com/apache/ignite-3/pull/3892 -- 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-21969 Extend test coverage for SQL F031-04(Basic schema manipulation. ALTER TABLE statement: ADD COLUMN clause) [ignite-3]

2024-06-10 Thread via GitHub
AMashenkov merged PR #3859: URL: https://github.com/apache/ignite-3/pull/3859 -- 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:

[PR] IGNITE-22302 Add resultSubscriber support to Data Streamer [ignite-3]

2024-06-10 Thread via GitHub
ptupitsyn opened a new pull request, #3900: URL: https://github.com/apache/ignite-3/pull/3900 (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-22445 Fix flaky ItCatalogDslTest [ignite-3]

2024-06-10 Thread via GitHub
Pochatkin merged PR #3897: URL: https://github.com/apache/ignite-3/pull/3897 -- 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-22319 Fixed node crashing if a snapshot restore cancelled due to network issues [ignite]

2024-06-10 Thread via GitHub
Vladsz83 commented on code in PR #11361: URL: https://github.com/apache/ignite/pull/11361#discussion_r1633130022 ## modules/core/src/test/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/IgniteSnapshotRestoreFromRemoteTest.java: ## @@ -323,6 +326,48 @@

Re: [PR] IGNITE-22319 Fixed node crashing if a snapshot restore cancelled due to network issues [ignite]

2024-06-10 Thread via GitHub
Vladsz83 commented on code in PR #11361: URL: https://github.com/apache/ignite/pull/11361#discussion_r1633123387 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/IgniteSnapshotManager.java: ## @@ -3895,20 +3879,22 @@ private Set

Re: [PR] IGNITE-22319 Fixed node crashing if a snapshot restore cancelled due to network issues [ignite]

2024-06-10 Thread via GitHub
NSAmelchev commented on code in PR #11361: URL: https://github.com/apache/ignite/pull/11361#discussion_r1633121390 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/IgniteSnapshotManager.java: ## @@ -3941,7 +3927,7 @@ else if (msg

[PR] IGNITE-21964 Extend test coverage for SQL E031-01(Identifiers. Delimited identifiers) [ignite-3]

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

[PR] IGNITE-21966: Extend test coverage for SQL E091-01(Set functions. AVG) [ignite-3]

2024-06-10 Thread via GitHub
lowka opened a new pull request, #3898: URL: https://github.com/apache/ignite-3/pull/3898 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-21966: Extend test coverage for SQL E091-01(Set functions. AVG) [ignite-3]

2024-06-10 Thread via GitHub
lowka closed pull request #3886: IGNITE-21966: Extend test coverage for SQL E091-01(Set functions. AVG) URL: https://github.com/apache/ignite-3/pull/3886 -- 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

[PR] IGNITE-22440 Add Maven command for building .zip files of Ignite extensions [ignite-extensions]

2024-06-10 Thread via GitHub
lordgarrish opened a new pull request, #274: URL: https://github.com/apache/ignite-extensions/pull/274 (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

Re: [PR] IGNITE-22319 Fixed node crashing if a snapshot restore cancelled due to network issues [ignite]

2024-06-10 Thread via GitHub
Vladsz83 commented on code in PR #11361: URL: https://github.com/apache/ignite/pull/11361#discussion_r1633018457 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/IgniteSnapshotManager.java: ## @@ -3941,7 +3927,7 @@ else if (msg

Re: [PR] IGNITE-22319 Fixed node crashing if a snapshot restore cancelled due to network issues [ignite]

2024-06-10 Thread via GitHub
Vladsz83 commented on code in PR #11361: URL: https://github.com/apache/ignite/pull/11361#discussion_r1632969420 ## modules/core/src/test/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/IgniteSnapshotRestoreFromRemoteTest.java: ## @@ -323,6 +326,46 @@

Re: [PR] IGNITE-22319 Fixed node crashing if a snapshot restore cancelled due to network issues [ignite]

2024-06-10 Thread via GitHub
Vladsz83 commented on code in PR #11361: URL: https://github.com/apache/ignite/pull/11361#discussion_r1632969178 ## modules/core/src/test/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/IgniteSnapshotRestoreFromRemoteTest.java: ## @@ -323,6 +326,46 @@

Re: [PR] IGNITE-22319 Fixed node crashing if a snapshot restore cancelled due to network issues [ignite]

2024-06-10 Thread via GitHub
NSAmelchev commented on code in PR #11361: URL: https://github.com/apache/ignite/pull/11361#discussion_r1632968947 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/IgniteSnapshotManager.java: ## @@ -3785,47 +3784,32 @@ public

Re: [PR] IGNITE-22319 Fixed node crashing if a snapshot restore cancelled due to network issues [ignite]

2024-06-10 Thread via GitHub
Vladsz83 commented on code in PR #11361: URL: https://github.com/apache/ignite/pull/11361#discussion_r1632916273 ## modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/snapshot/IgniteSnapshotManager.java: ## @@ -3785,47 +3784,32 @@ public

Re: [PR] Bump org.rocksdb:rocksdbjni from 9.1.1 to 9.2.1 [ignite-3]

2024-06-10 Thread via GitHub
ptupitsyn merged PR #3890: URL: https://github.com/apache/ignite-3/pull/3890 -- 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-20450 Add failure processor and failure handlers configuration. [ignite-3]

2024-06-10 Thread via GitHub
sk0x50 merged PR #3862: URL: https://github.com/apache/ignite-3/pull/3862 -- 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-20650 Fix case-sensitive field names resolution [ignite-3]

2024-06-10 Thread via GitHub
korlov42 commented on code in PR #3850: URL: https://github.com/apache/ignite-3/pull/3850#discussion_r1632679773 ## modules/api/src/main/java/org/apache/ignite/table/Tuple.java: ## @@ -193,7 +195,9 @@ static boolean equals(Tuple firstTuple, Tuple secondTuple) { * Gets a

Re: [PR] Bump org.rocksdb:rocksdbjni from 9.1.1 to 9.2.1 [ignite-3]

2024-06-10 Thread via GitHub
ptupitsyn commented on PR #3890: URL: https://github.com/apache/ignite-3/pull/3890#issuecomment-2157461608 @dependabot rebase -- 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] Bump com.google.guava:guava from 33.2.0-jre to 33.2.1-jre [ignite-3]

2024-06-10 Thread via GitHub
ptupitsyn merged PR #3889: URL: https://github.com/apache/ignite-3/pull/3889 -- 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] Bump org.jctools:jctools-core from 4.0.3 to 4.0.5 [ignite-3]

2024-06-10 Thread via GitHub
ptupitsyn merged PR #3882: URL: https://github.com/apache/ignite-3/pull/3882 -- 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-22244 Do not format configuration string [ignite-3]

2024-06-07 Thread via GitHub
PakhomovAlexander merged PR #3893: URL: https://github.com/apache/ignite-3/pull/3893 -- 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-22384 Replace Conan package manager with direct dependency management using cmake [ignite-3]

2024-06-07 Thread via GitHub
isapego merged PR #3851: URL: https://github.com/apache/ignite-3/pull/3851 -- 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:

[PR] IGNITE-22445 Fix flaky ItCatalogDslTest [ignite-3]

2024-06-07 Thread via GitHub
valepakh opened a new pull request, #3897: URL: https://github.com/apache/ignite-3/pull/3897 https://issues.apache.org/jira/browse/IGNITE-22445 -- 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-21868 Moved the sql RO inflights handling from SqlQueryProcess… [ignite-3]

2024-06-07 Thread via GitHub
korlov42 merged PR #3511: URL: https://github.com/apache/ignite-3/pull/3511 -- 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-22296 Rename cluster init options in CLI [ignite-3]

2024-06-07 Thread via GitHub
PakhomovAlexander merged PR #3891: URL: https://github.com/apache/ignite-3/pull/3891 -- 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:

[PR] IGNITE-21957 Sql. Extend test coverage for SQL E021-10 (Character string types. Implicit casting among the character string types) [ignite-3]

2024-06-07 Thread via GitHub
zstan opened a new pull request, #3896: URL: https://github.com/apache/ignite-3/pull/3896 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-22423 Don't use kill -9 for ignite3db [ignite-3]

2024-06-07 Thread via GitHub
Pochatkin commented on code in PR #3894: URL: https://github.com/apache/ignite-3/pull/3894#discussion_r1631133516 ## packaging/db/zip/linux/ignite3db: ## @@ -59,7 +59,20 @@ start() { } stop() { - kill -9 "$(cat ${IGNITE_HOME}/pid)" + ignite_pid = "$(cat

Re: [PR] IGNITE-22414 Fix cluster status command for not initialized cluster [ignite-3]

2024-06-07 Thread via GitHub
PakhomovAlexander merged PR #3879: URL: https://github.com/apache/ignite-3/pull/3879 -- 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-22384 Replace Conan package manager with direct dependency management using cmake [ignite-3]

2024-06-07 Thread via GitHub
isapego commented on code in PR #3851: URL: https://github.com/apache/ignite-3/pull/3851#discussion_r1631119607 ## modules/platforms/cpp/DEVNOTES.md: ## @@ -2,66 +2,35 @@ ### Prerequisites - * C++ compiler supporting C++ 17 * One of the build systems: make, ninja, MS

[PR] IGNITE-22439 Add tests for duplicate keys in Calcite [ignite]

2024-06-07 Thread via GitHub
lordgarrish opened a new pull request, #11380: URL: https://github.com/apache/ignite/pull/11380 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-21868 Moved the sql RO inflights handling from SqlQueryProcess… [ignite-3]

2024-06-07 Thread via GitHub
denis-chudov commented on code in PR #3511: URL: https://github.com/apache/ignite-3/pull/3511#discussion_r1631109543 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/tx/QueryTransactionWrapperImpl.java: ## @@ -42,6 +56,10 @@ public InternalTransaction

Re: [PR] IGNITE-22423 Don't use kill -9 for ignite3db [ignite-3]

2024-06-07 Thread via GitHub
Lozmanov commented on code in PR #3894: URL: https://github.com/apache/ignite-3/pull/3894#discussion_r1631092166 ## packaging/db/zip/linux/ignite3db: ## @@ -59,7 +59,19 @@ start() { } stop() { - kill -9 "$(cat ${IGNITE_HOME}/pid)" + kill -15 "$(cat ${IGNITE_HOME}/pid)" +

Re: [PR] IGNITE-21868 Moved the sql RO inflights handling from SqlQueryProcess… [ignite-3]

2024-06-07 Thread via GitHub
denis-chudov commented on code in PR #3511: URL: https://github.com/apache/ignite-3/pull/3511#discussion_r1631109543 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/tx/QueryTransactionWrapperImpl.java: ## @@ -42,6 +56,10 @@ public InternalTransaction

Re: [PR] IGNITE-21868 Moved the sql RO inflights handling from SqlQueryProcess… [ignite-3]

2024-06-07 Thread via GitHub
denis-chudov commented on code in PR #3511: URL: https://github.com/apache/ignite-3/pull/3511#discussion_r1631106581 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/tx/ScriptTransactionWrapperImpl.java: ## @@ -184,6 +191,10 @@ private void completeTx()

Re: [PR] IGNITE-22443 Fix sporadic configuration tests failures [ignite-3]

2024-06-07 Thread via GitHub
ibessonov commented on code in PR #3895: URL: https://github.com/apache/ignite-3/pull/3895#discussion_r1631103525 ## modules/configuration/src/testFixtures/java/org/apache/ignite/internal/configuration/TestConfigurationChanger.java: ## @@ -51,6 +53,13 @@ public

Re: [PR] IGNITE-22423 Don't use kill -9 for ignite3db [ignite-3]

2024-06-07 Thread via GitHub
Lozmanov commented on code in PR #3894: URL: https://github.com/apache/ignite-3/pull/3894#discussion_r1631103321 ## packaging/db/zip/linux/ignite3db: ## @@ -59,7 +59,20 @@ start() { } stop() { - kill -9 "$(cat ${IGNITE_HOME}/pid)" + ignite_pid = "$(cat

Re: [PR] IGNITE-22423 Don't use kill -9 for ignite3db [ignite-3]

2024-06-07 Thread via GitHub
Lozmanov commented on code in PR #3894: URL: https://github.com/apache/ignite-3/pull/3894#discussion_r1631101524 ## packaging/db/zip/linux/ignite3db: ## @@ -59,7 +59,20 @@ start() { } stop() { - kill -9 "$(cat ${IGNITE_HOME}/pid)" + ignite_pid = "$(cat

Re: [PR] IGNITE-22423 Don't use kill -9 for ignite3db [ignite-3]

2024-06-07 Thread via GitHub
Lozmanov commented on code in PR #3894: URL: https://github.com/apache/ignite-3/pull/3894#discussion_r1631097601 ## packaging/db/zip/linux/ignite3db: ## @@ -59,7 +59,20 @@ start() { } stop() { - kill -9 "$(cat ${IGNITE_HOME}/pid)" + ignite_pid = "$(cat

Re: [PR] IGNITE-22423 Don't use kill -9 for ignite3db [ignite-3]

2024-06-07 Thread via GitHub
Lozmanov commented on code in PR #3894: URL: https://github.com/apache/ignite-3/pull/3894#discussion_r1631095878 ## packaging/db/zip/linux/ignite3db: ## @@ -59,7 +59,20 @@ start() { } stop() { - kill -9 "$(cat ${IGNITE_HOME}/pid)" + ignite_pid = "$(cat

Re: [PR] IGNITE-22443 Fix sporadic configuration tests failures [ignite-3]

2024-06-07 Thread via GitHub
Phillippko commented on code in PR #3895: URL: https://github.com/apache/ignite-3/pull/3895#discussion_r1631094948 ## modules/configuration/src/testFixtures/java/org/apache/ignite/internal/configuration/TestConfigurationChanger.java: ## @@ -51,6 +53,13 @@ public

Re: [PR] IGNITE-22423 Don't use kill -9 for ignite3db [ignite-3]

2024-06-07 Thread via GitHub
Lozmanov commented on code in PR #3894: URL: https://github.com/apache/ignite-3/pull/3894#discussion_r1631092166 ## packaging/db/zip/linux/ignite3db: ## @@ -59,7 +59,19 @@ start() { } stop() { - kill -9 "$(cat ${IGNITE_HOME}/pid)" + kill -15 "$(cat ${IGNITE_HOME}/pid)" +

Re: [PR] IGNITE-22423 Don't use kill -9 for ignite3db [ignite-3]

2024-06-07 Thread via GitHub
Pochatkin commented on code in PR #3894: URL: https://github.com/apache/ignite-3/pull/3894#discussion_r1631089997 ## packaging/db/zip/linux/ignite3db: ## @@ -59,7 +59,19 @@ start() { } stop() { - kill -9 "$(cat ${IGNITE_HOME}/pid)" + kill -15 "$(cat ${IGNITE_HOME}/pid)"

Re: [PR] IGNITE-22367 Create index metadata storage [ignite-3]

2024-06-07 Thread via GitHub
tkalkirill merged PR #3887: URL: https://github.com/apache/ignite-3/pull/3887 -- 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:

[PR] IGNITE-22443 Fix sporadic configuration tests failures [ignite-3]

2024-06-07 Thread via GitHub
ibessonov opened a new pull request, #3895: URL: https://github.com/apache/ignite-3/pull/3895 https://issues.apache.org/jira/browse/IGNITE-22443 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-22423 Don't use kill -9 for ignite3db [ignite-3]

2024-06-07 Thread via GitHub
Lozmanov commented on code in PR #3894: URL: https://github.com/apache/ignite-3/pull/3894#discussion_r1631085444 ## packaging/db/zip/linux/ignite3db: ## @@ -59,7 +59,19 @@ start() { } stop() { - kill -9 "$(cat ${IGNITE_HOME}/pid)" + kill -15 "$(cat ${IGNITE_HOME}/pid)" +

Re: [PR] IGNITE-21868 Moved the sql RO inflights handling from SqlQueryProcess… [ignite-3]

2024-06-07 Thread via GitHub
denis-chudov commented on code in PR #3511: URL: https://github.com/apache/ignite-3/pull/3511#discussion_r1631084161 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/tx/QueryTransactionContextImpl.java: ## @@ -50,13 +58,23 @@ public

Re: [PR] IGNITE-20503 Sql. Support big clusters by mapping service [ignite-3]

2024-06-07 Thread via GitHub
korlov42 commented on code in PR #3846: URL: https://github.com/apache/ignite-3/pull/3846#discussion_r1631071669 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/mapping/smallcluster/AbstractTarget.java: ## @@ -34,7 +34,7 @@ * Children of this

Re: [PR] IGNITE-22423 Don't use kill -9 for ignite3db [ignite-3]

2024-06-07 Thread via GitHub
Lozmanov commented on code in PR #3894: URL: https://github.com/apache/ignite-3/pull/3894#discussion_r1631085067 ## packaging/db/zip/linux/ignite3db: ## @@ -59,7 +59,19 @@ start() { } stop() { - kill -9 "$(cat ${IGNITE_HOME}/pid)" + kill -15 "$(cat ${IGNITE_HOME}/pid)"

Re: [PR] IGNITE-21868 Moved the sql RO inflights handling from SqlQueryProcess… [ignite-3]

2024-06-07 Thread via GitHub
denis-chudov commented on code in PR #3511: URL: https://github.com/apache/ignite-3/pull/3511#discussion_r1631084161 ## modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/tx/QueryTransactionContextImpl.java: ## @@ -50,13 +58,23 @@ public

[PR] IGNITE-22423 Don't use kill -9 for ignite3db [ignite-3]

2024-06-07 Thread via GitHub
Pochatkin opened a new pull request, #3894: URL: https://github.com/apache/ignite-3/pull/3894 https://issues.apache.org/jira/browse/IGNITE-22423 -- 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

  1   2   3   4   5   6   7   8   9   10   >