[PR] IGNITE-22278 Sql. Refactor base and planning contexts [ignite-3]

2024-05-17 Thread via GitHub
korlov42 opened a new pull request, #3787: URL: https://github.com/apache/ignite-3/pull/3787 https://issues.apache.org/jira/browse/IGNITE-22278 This patch refactors contexts used within sql engine: 1) BaseQueryContext is removed. Instead lightweight SqlOperationContext is introduce

[PR] IGNITE-19103: Sql. Change implicit primary key column's type to UUID. [ignite-3]

2024-05-17 Thread via GitHub
lowka opened a new pull request, #3786: URL: https://github.com/apache/ignite-3/pull/3786 --- 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] WIP [ignite-3]

2024-05-17 Thread via GitHub
korlov42 opened a new pull request, #3785: URL: https://github.com/apache/ignite-3/pull/3785 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-22118 Explicitly specify pool to execute start/stop on IgniteComponent [ignite-3]

2024-05-17 Thread via GitHub
Cyrill opened a new pull request, #3784: URL: https://github.com/apache/ignite-3/pull/3784 https://issues.apache.org/jira/browse/IGNITE-22118 -- 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 s

[PR] IGNITE-22099 Delete TransactionDuplicateKeyException [ignite]

2024-05-17 Thread via GitHub
lordgarrish opened a new pull request, #11354: URL: https://github.com/apache/ignite/pull/11354 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-22149 Java Thin client table partition API [ignite-3]

2024-05-17 Thread via GitHub
Pochatkin commented on code in PR #3764: URL: https://github.com/apache/ignite-3/pull/3764#discussion_r1604803074 ## modules/client-common/src/main/java/org/apache/ignite/internal/client/proto/ClientOp.java: ## @@ -170,4 +170,10 @@ public class ClientOp { /** Execute SQL

Re: [PR] IGNITE-22149 Java Thin client table partition API [ignite-3]

2024-05-17 Thread via GitHub
Pochatkin commented on code in PR #3764: URL: https://github.com/apache/ignite-3/pull/3764#discussion_r1604803683 ## modules/client-handler/src/main/java/org/apache/ignite/client/handler/requests/table/partition/ClientPartitionPrimaryGetRequest.java: ## @@ -0,0 +1,76 @@ +/* + *

Re: [PR] IGNITE-22268 Add checkstyle rule for package names [ignite-3]

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

[PR] IGNITE-22268 Add checkstyle rule for package names [ignite-3]

2024-05-17 Thread via GitHub
valepakh opened a new pull request, #3783: URL: https://github.com/apache/ignite-3/pull/3783 https://issues.apache.org/jira/browse/IGNITE-22268 -- 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-21979: Extend test coverage for SQL F781(Self-referencing operations) [ignite-3]

2024-05-17 Thread via GitHub
ygerzhedovich merged PR #3759: URL: https://github.com/apache/ignite-3/pull/3759 -- 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-22120] Sql. Possibility set zero length for character string type [ignite-3]

2024-05-17 Thread via GitHub
lowka opened a new pull request, #3782: URL: https://github.com/apache/ignite-3/pull/3782 --- 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-22124 Java thin: compute MapReduce impl [ignite-3]

2024-05-17 Thread via GitHub
vadimkolodingg commented on code in PR #3744: URL: https://github.com/apache/ignite-3/pull/3744#discussion_r1604632615 ## modules/client-handler/src/main/java/org/apache/ignite/client/handler/requests/compute/ClientComputeExecuteMapReduceRequest.java: ## @@ -0,0 +1,102 @@ +/* +

Re: [PR] IGNITE-22124 Java thin: compute MapReduce impl [ignite-3]

2024-05-17 Thread via GitHub
vadimkolodingg commented on code in PR #3744: URL: https://github.com/apache/ignite-3/pull/3744#discussion_r1604631231 ## modules/client/src/main/java/org/apache/ignite/internal/client/compute/ClientCompute.java: ## @@ -442,6 +474,31 @@ private static SubmitResult unpackSubmitR

Re: [PR] IGNITE-22124 Java thin: compute MapReduce impl [ignite-3]

2024-05-17 Thread via GitHub
vadimkolodingg commented on code in PR #3744: URL: https://github.com/apache/ignite-3/pull/3744#discussion_r1604626243 ## modules/client/src/main/java/org/apache/ignite/internal/client/ClientUtils.java: ## @@ -213,6 +213,7 @@ public static ClientOperationType opCodeToClientOper

Re: [PR] IGNITE-22124 Java thin: compute MapReduce impl [ignite-3]

2024-05-17 Thread via GitHub
vadimkolodingg commented on code in PR #3744: URL: https://github.com/apache/ignite-3/pull/3744#discussion_r1604623359 ## modules/client-handler/src/main/java/org/apache/ignite/client/handler/requests/compute/ClientComputeExecuteMapReduceRequest.java: ## @@ -0,0 +1,102 @@ +/* +

Re: [PR] IGNITE-22124 Java thin: compute MapReduce impl [ignite-3]

2024-05-17 Thread via GitHub
vadimkolodingg commented on code in PR #3744: URL: https://github.com/apache/ignite-3/pull/3744#discussion_r1604622481 ## modules/client-handler/src/main/java/org/apache/ignite/client/handler/requests/compute/ClientComputeExecuteMapReduceRequest.java: ## @@ -0,0 +1,102 @@ +/* +

[I] CPU goes crazy in ignite server [ignite]

2024-05-17 Thread via GitHub
billleyn023 opened a new issue, #11353: URL: https://github.com/apache/ignite/issues/11353 Greetings, Our system works well before but recently CPU goes high and never get down. The architecture is like that, we have three clients, five servers whose CPU have 16 cores per each.

Re: [PR] IGNITE-22124 Java thin: compute MapReduce impl [ignite-3]

2024-05-17 Thread via GitHub
ptupitsyn commented on code in PR #3744: URL: https://github.com/apache/ignite-3/pull/3744#discussion_r1604481973 ## modules/client-handler/src/main/java/org/apache/ignite/client/handler/requests/compute/ClientComputeExecuteMapReduceRequest.java: ## @@ -0,0 +1,102 @@ +/* + * Lic