Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IGNITE-24197 .NET: Unify Сompute API #5050

Open
wants to merge 35 commits into
base: main
Choose a base branch
from

Conversation

ptupitsyn
Copy link
Contributor

@ptupitsyn ptupitsyn commented Jan 15, 2025

Use Target and Execution objects in all Compute API methods.

  • Add IBroadcastJobTarget
  • Add IBroadcastExecution
  • Remove CancelAsync from JobExecution and TaskExecution (use CancellationToken for cancellation - to be added separately)

Reflects Java API changes from #5016

valepakh and others added 30 commits January 9, 2025 21:07
…iled future, compare node names instead of ClusterNode, fix matcher
# Conflicts:
#	modules/client-handler/src/main/java/org/apache/ignite/client/handler/requests/compute/ClientComputeExecuteColocatedRequest.java
#	modules/client-handler/src/main/java/org/apache/ignite/client/handler/requests/compute/ClientComputeExecuteRequest.java
#	modules/client/src/test/java/org/apache/ignite/client/fakes/FakeCompute.java
#	modules/compute/src/integrationTest/java/org/apache/ignite/internal/compute/ItComputeBaseTest.java
#	modules/compute/src/integrationTest/java/org/apache/ignite/internal/compute/ItComputeTestClient.java
#	modules/compute/src/integrationTest/java/org/apache/ignite/internal/compute/ItComputeTestEmbedded.java
#	modules/compute/src/main/java/org/apache/ignite/internal/compute/ComputeComponentImpl.java
#	modules/compute/src/main/java/org/apache/ignite/internal/compute/DelegatingJobExecution.java
#	modules/compute/src/main/java/org/apache/ignite/internal/compute/IgniteComputeImpl.java
#	modules/compute/src/main/java/org/apache/ignite/internal/compute/executor/ComputeExecutorImpl.java
#	modules/compute/src/test/java/org/apache/ignite/internal/compute/IgniteComputeImplTest.java
#	modules/runner/src/integrationTest/java/org/apache/ignite/internal/runner/app/compute/ItEmbeddedMarshallingTest.java
# Conflicts:
#	modules/api/src/main/java/org/apache/ignite/compute/IgniteCompute.java
#	modules/client-common/src/main/java/org/apache/ignite/internal/compute/BroadcastJobExecutionImpl.java
#	modules/client/src/main/java/org/apache/ignite/internal/client/compute/ClientCompute.java
#	modules/client/src/test/java/org/apache/ignite/client/fakes/FakeCompute.java
#	modules/compute/src/integrationTest/java/org/apache/ignite/internal/compute/ItComputeBaseTest.java
#	modules/compute/src/integrationTest/java/org/apache/ignite/internal/compute/ItExecutionsCleanerTest.java
#	modules/compute/src/integrationTest/java/org/apache/ignite/internal/compute/ItMapReduceTest.java
#	modules/compute/src/integrationTest/java/org/apache/ignite/internal/compute/ItWorkerShutdownTest.java
#	modules/compute/src/main/java/org/apache/ignite/internal/compute/AntiHijackIgniteCompute.java
#	modules/compute/src/main/java/org/apache/ignite/internal/compute/IgniteComputeImpl.java
#	modules/runner/src/integrationTest/java/org/apache/ignite/internal/runner/app/client/ItAbstractThinClientTest.java
#	modules/runner/src/integrationTest/java/org/apache/ignite/internal/runner/app/client/ItThinClientComputeTest.java
#	modules/runner/src/main/java/org/apache/ignite/internal/restart/RestartProofIgniteCompute.java
#	modules/sql-engine/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItComputeSystemViewTest.java
@ptupitsyn ptupitsyn self-assigned this Jan 15, 2025
@ptupitsyn ptupitsyn requested a review from isapego January 15, 2025 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants