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.

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

[Bug]: TransactionContextStore is not maintained when invokeMethodAsyncSequentially() is called #40075

Closed
kaneeldias opened this issue Apr 3, 2023 · 0 comments · Fixed by #40235
Closed

[Bug]: TransactionContextStore is not maintained when invokeMethodAsyncSequentially() is called #40075

kaneeldias opened this issue Apr 3, 2023 · 0 comments · Fixed by #40235
Assignees
@pcnfernando
Labels
Reason/EngineeringMistake The issue occurred due to a mistake made in the past. Team/jBallerina All the issues related to BIR, JVM backend code generation and runtime Type/Bug
Milestone
2201.6.0

Comments

@kaneeldias
Copy link
Contributor

kaneeldias commented Apr 3, 2023 •
edited
Loading

Description

When runtime.invokeMethodAsyncSequentially() is called with an existing TransactionContext passed as a property, in the newly created Strand, while the currentTrxContext is populated as expected the transactionContextStore is empty.

However, in the TransactionContext within the globalProps inside the currentTrxContext, contains the correct context store.

image

The issue seems to be when a new Strand is created, the branched Context doesn't maintain the contextStore.

When a new TransactionLocalContext is initialized, the transactionContextStore is not maintained.

Steps to Reproduce

Update 5

Affected Version(s)

Update 5

OS, DB, other environment details and versions

No response

Related area

-> Runtime

Related issue(s) (optional)

ballerina-platform/ballerina-library#4270

Suggested label(s) (optional)

No response

Suggested assignee(s) (optional)

No response
@kaneeldias kaneeldias changed the title [Bug]: Transaction Context Store is not maintained when `invokeMethodAsyncSequentially [Bug]: Transaction Context Store is not maintained when invokeMethodAsyncSequentially() is called Apr 3, 2023
@ballerina-bot ballerina-bot added the Team/jBallerina All the issues related to BIR, JVM backend code generation and runtime label Apr 3, 2023
@kaneeldias kaneeldias changed the title [Bug]: Transaction Context Store is not maintained when invokeMethodAsyncSequentially() is called [Bug]: TransactionContextStore is not maintained when invokeMethodAsyncSequentially() is called Apr 3, 2023
@kaneeldias kaneeldias mentioned this issue Apr 3, 2023
Closed
@pcnfernando pcnfernando added this to the 2201.6.0 milestone Apr 3, 2023
@pcnfernando pcnfernando mentioned this issue Apr 6, 2023
Merged
13 tasks
@pcnfernando pcnfernando mentioned this issue Apr 17, 2023
Merged
13 tasks
@pcnfernando pcnfernando added the Reason/EngineeringMistake The issue occurred due to a mistake made in the past. label Apr 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pcnfernando pcnfernando

Labels
Reason/EngineeringMistake The issue occurred due to a mistake made in the past. Team/jBallerina All the issues related to BIR, JVM backend code generation and runtime Type/Bug
Projects
Ballerina Team Main Board
Archived in project
Milestone
2201.6.0
Development

Successfully merging a pull request may close this issue.

[master] Populate transaction branch with connection contexts pcnfernando/ballerina-lang
3 participants
@kaneeldias @pcnfernando @ballerina-bot