-
Notifications
You must be signed in to change notification settings - Fork 17k
Add Coordinator Layer and Java Coordinator #65958
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
Draft
jason810496
wants to merge
51
commits into
apache:main
Choose a base branch
from
astronomer:task-sdk/feature/coordinator-interface
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
Show all changes
51 commits
Select commit
Hold shift + click to select a range
3c9a985
Add Task Coordinators and Dag File Processor
jason810496 c3862e6
Add initial Java provider for Apache Airflow
jason810496 32276f0
Add common selector loop utilities for socket I/O handling for subpro…
jason810496 5167db1
Implement Java DAG file processor with TCP communication bridge
jason810496 f9c67b5
Make JavaDagFileProcessor.can_handle aware of jar file content
jason810496 6825e04
Fix java process startup issue
jason810496 564a0ab
Fix sockets bidning
jason810496 83d0a3d
Refactor Java DAG file processor to use selector-based I/O multiplexi…
jason810496 adfb68e
Add BaseLocaleCoordinator for non-Python DAG file processing and task…
jason810496 9c61249
Implement JavaLocaleCoordinator
jason810496 d7c9e2d
Add Java task coordinator and entrypoint for locale-specific execution
jason810496 c2ec72a
Refactor Java provider to with generic process coordinators and updat…
jason810496 5fc2ae0
Fix Coordinator by getting the correct dag bundle and dag path
jason810496 bb6cdc1
Make @task.stub(language=java) works
jason810496 343b302
Make coordinator respect Jar bundle based on TI workload type
jason810496 4022ce7
Add java_sdk_setup script for Breeze
jason810496 4a63d24
Add get_code_from_file interface for BaseLocaleCoordinator
jason810496 08dea47
Fix the 'Pure Java Dag' disappear in metadata DB issue
jason810496 cc5be91
Refactor process coordinators to runtime coordinators
jason810496 0e796a0
Rename stub operator language field as sdk
jason810496 94eee15
Rename languages.java provider to sdk.java
jason810496 38780e7
Add unit tests for socket handling and selector loop functionality
jason810496 02b22f1
Move TaskInstanceDTO to share to make task_runner retrieve TI.queue
jason810496 869b931
Add [workers/queue_to_runtime_mapping]
jason810496 52f0f4c
Remove the sdk field from stub operator and respect [workers/queue_to…
jason810496 96a4fc5
Rename `[workers] queue_to_runtime_mapping` to `[sdk] queue_to_sdk`
jason810496 7096500
Simplify coordinator-related names (#1569)
uranusjr 8e56e24
CI: Add mypy and unit tests for shared/workloads
jason810496 8f9c4e4
CI: Fix DB migration and breeze images
jason810496 913eab7
CI: Fix failing items
jason810496 26d3802
CI: Fix failing items
jason810496 95d40f1
CI: Add compat for create_runtime_ti pytest fixture
jason810496 68c72b2
CI: Fix Java provider test to include configuration options
jason810496 6f7b570
CI: Fix self-review nits
jason810496 82db9e7
Revert MappedOperator change
jason810496 e6b4c2d
CI: Fix failing items
jason810496 e5b24bb
CI: Fix Task SDK test_task_runner failures using TaskInstanceDTO
jason810496 a0dc159
CI: Skip non-JAR paths in JavaCoordinator.can_handle_dag_file
jason810496 bee63fc
CI: Drop literal Example: line from queue_to_sdk config description
jason810496 5c586dd
CI: Skip sdk-java provider in compat tests for older Airflow
jason810496 c40cf3e
CI: Fix MyPy Liskov violation in JavaCoordinator.task_execution_cmd
jason810496 8b2a565
CI: Fix sdk-java docs build warnings
jason810496 b8f6f4f
CI: Update SDK Java configuration and documentation references
jason810496 518b876
CI: Update map_index handling and add fixture to restore process cont…
jason810496 2a5d6f6
CI: Refactor map_index handling and update time travel decorators for…
jason810496 8fce19f
CI: Replace TaskInstance with TaskInstanceDTO and add additional para…
jason810496 23e3734
CI: Update test cases to use -1 for map_index and add additional para…
jason810496 53b83f0
Remove shared workloads dependency and refactor TaskInstanceDTO usage
jason810496 3d719d1
Move sdk.java out of provider as apache-airflow-coordinators-java dis…
jason810496 ed38952
Enhance documentation for BaseCoordinator lifecycle methods and IPC m…
jason810496 bc32136
CI: Fix docs spellcheck and code-block indent for sdk coordinators ex…
jason810496 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -39,6 +39,7 @@ | |
| !task-sdk/ | ||
| !airflow-ctl/ | ||
| !go-sdk/ | ||
| !sdk/ | ||
|
|
||
| # Add all "test" distributions | ||
| !tests | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Go SDK would not be listed here?
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Once the go-sdk adapt the coordinator interface as a provider, I will update the description here to avoid the confusion.