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

Data Import: Database transaction not releasing correctly #172

Closed
DnOberon opened this issue Mar 10, 2021 · 0 comments · Fixed by #170
Closed

Data Import: Database transaction not releasing correctly #172

DnOberon opened this issue Mar 10, 2021 · 0 comments · Fixed by #170
Assignees
@jerenmb
Labels
bug Something isn't working High Priority

Comments

@DnOberon
Copy link
Collaborator

Bug Description

The process method inside src/data_processing/processing.ts is attempting to rollback and/or complete the database transaction passed to it. In reality, the caller of process should be in charge of rollbacking/completing database transactions - or the rollback transaction method should stop releasing a client.

Steps to Reproduce

Attempt to process data, you'll find yourself getting either locking errors or double release errors.

Impact

High impact, anyone attempting to process data will run into this issue.
@DnOberon DnOberon added bug Something isn't working High Priority labels Mar 10, 2021
@DnOberon DnOberon mentioned this issue Mar 30, 2021
Merged
33 tasks
DnOberon added a commit that referenced this issue Jan 6, 2023
@DnOberon
Merge pull request #172 from Digital-Engineering/development
ade7d83 
workflow change again
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jerenmb jerenmb

Labels
bug Something isn't working High Priority
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Domain, Repository, Mapper Refactor idaholab/Deep-Lynx
2 participants
@DnOberon @jerenmb