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

Send worker changes to master #42294

Merged
merged 137 commits into from
Mar 27, 2024
Merged

Send worker changes to master #42294

merged 137 commits into from
Mar 27, 2024

Conversation

lochana-chathura
Copy link
Member

@lochana-chathura lochana-chathura commented Mar 11, 2024

Purpose

$subject. This PR adds support for the following features,

  • Alternate receive action support
  • Multiple receive action support
  • Conditional send support
  • Named workers with on-fail-clause support

Fixes #41699 Fixes #41953 Fixes #41968 Fixes #42004

This PR includes collective changes from the subsequent PRs.

Approach

n/a

Samples

n/a

Remarks

Sequence diagram behavior: #42151 (comment)

Check List

  • Read the Contributing Guide
  • Updated Change Log
  • Checked Tooling Support (#)
  • Added necessary tests
    • Unit Tests
    • Spec Conformance Tests
    • Integration Tests
    • Ballerina By Example Tests
  • Increased Test Coverage
  • Added necessary documentation
    • API documentation
    • Module documentation in Module.md files
    • Ballerina By Examples

lochana-chathura and others added 30 commits November 14, 2023 10:01
Add parsing support for new worker related syntax
Each pair of send-receive is associated with an index called eventIndex
Implement new worker semantics - Part I
[Worker Change] Fix failing worker `on fail` parser test
@chiranSachintha chiranSachintha self-requested a review March 18, 2024 08:57
…nal/scheduling/WDChannels.java

Co-authored-by: Sasindu Alahakoon <47134066+SasinduDilshara@users.noreply.github.com>
@HindujaB HindujaB mentioned this pull request Mar 20, 2024
13 tasks
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)
5.3% Duplication on New Code (required ≤ 3%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

Copy link
Contributor

@HindujaB HindujaB left a comment

Choose a reason for hiding this comment

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

LGTM

@lochana-chathura lochana-chathura merged commit 505f257 into master Mar 27, 2024
18 of 19 checks passed
@lochana-chathura lochana-chathura deleted the worker_change branch March 27, 2024 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team/CompilerFE All issues related to Language implementation and Compiler, this exclude run times. Team/jBallerina All the issues related to BIR, JVM backend code generation and runtime
Projects
None yet
9 participants