Skip to content

Core: add support for Pipeline/Batch #3521

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Apr 6, 2025
Merged

Core: add support for Pipeline/Batch #3521

merged 4 commits into from
Apr 6, 2025

Conversation

@shohamazon
Copy link
Collaborator

@shohamazon shohamazon commented Apr 3, 2025 •
edited
Loading

This PR pushes the pipelines-target-branch into main, as it contains the core changes for pipeline support in glide.

The PR includes the changes presented in #2954 #3325 #3211 #3359

Issue link

This Pull Request is linked to issue (URL): #3542

Checklist

Before submitting the PR make sure the following are checked:

  • This Pull Request is related to one issue.
  • Commit message has a detailed description of what changed and why.
  • Tests are added or updated.
  • CHANGELOG.md and documentation files are updated.
  • Destination branch is correct - main or release
  • Create merge commit if merging release branch into main, squash otherwise.

@shohamazon shohamazon requested a review from a team as a code owner April 3, 2025 19:47
@shohamazon shohamazon force-pushed the pipelines-target-branch branch from e09e7c6 to d100993 Compare April 3, 2025 19:52
@shohamazon shohamazon changed the title Pipelines target branch Core: add support for Pipeline/Batch Apr 3, 2025
@shohamazon shohamazon self-assigned this Apr 3, 2025
@shohamazon shohamazon added Rust core Core changes 🪐 Used to label a PR as PR with significant changes that should trigger a full matrix tests. 2_0_candidate Candidate task for version 2.0 labels Apr 3, 2025
Yury-Fridlyand
Yury-Fridlyand reviewed Apr 3, 2025
View reviewed changes
Copy link

@Yury-Fridlyand Yury-Fridlyand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a changelog

shohamazon and others added 4 commits April 6, 2025 08:16
@shohamazon
Core: Add initial Pipeline support (#2954)
6d2bbaa 
---------
Signed-off-by: Shoham Elias <shohame@amazon.com>
@shohamazon
Move ServerError into Value
aa3ab80 
Signed-off-by: Shoham Elias <shohame@amazon.com>
@shohamazon
Refactor command handling to use Arc<Cmd> for improved memory manag
0016c9d 
Signed-off-by: Shoham Elias <shohame@amazon.com>
@shohamazon
Pipeline: add support for error handling (#3359)
758cef8 
---------

Signed-off-by: Shoham Elias <shohame@amazon.com>
@shohamazon shohamazon force-pushed the pipelines-target-branch branch from d100993 to 758cef8 Compare April 6, 2025 08:16
@shohamazon shohamazon mentioned this pull request Apr 6, 2025
Closed
6 tasks
barshaul
barshaul approved these changes Apr 6, 2025
View reviewed changes
Copy link
Collaborator

@barshaul barshaul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀 🚀 🚀 🚀 🚀

@shohamazon shohamazon merged commit aef332a into main Apr 6, 2025
62 checks passed
@Yury-Fridlyand Yury-Fridlyand deleted the pipelines-target-branch branch April 7, 2025 15:20
@Yury-Fridlyand Yury-Fridlyand added the breaking 💔 breaking changes label Apr 7, 2025
This was referenced Apr 15, 2025
Merged
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@barshaul barshaul barshaul approved these changes

+1 more reviewer

@Yury-Fridlyand Yury-Fridlyand Yury-Fridlyand left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@shohamazon shohamazon

Labels

2_0_candidate Candidate task for version 2.0 breaking 💔 breaking changes Core changes 🪐 Used to label a PR as PR with significant changes that should trigger a full matrix tests.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@shohamazon @barshaul @Yury-Fridlyand