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

Transaction bug - close context in finally block (#845) #852

Merged
merged 5 commits into from
Jan 16, 2025

Conversation

milanmajchrak
Copy link
Collaborator

Phases MP MM MB MR JM Total
ETA 0 0 0 0 0 0
Developing 0 0 0 0 0 0
Review 0 0 0 0 0 0
Total - - - - - 0
ETA est. 0
ETA cust. - - - - - 0

Problem description

(Write here, if some unexpected problems occur during solving issues. Erase it, when it is not needed.)

* Logging hibernate statistics doesn't work as expected.

* Undo connection pool timeout contants

* Removed a huge amount of log.info

* Added abort to try catch blocks to check if it will solve IT

* Call aborting the context in the finally block of the StatelessAuthenticationFilter

---------

Co-authored-by: jm <jm@maz>
@milanmajchrak milanmajchrak changed the base branch from dtq-dev to ufal/be-user_metadata-wrong-import January 16, 2025 14:23
@milanmajchrak milanmajchrak changed the base branch from ufal/be-user_metadata-wrong-import to dtq-dev January 16, 2025 14:23
@milanmajchrak milanmajchrak self-assigned this Jan 16, 2025
@milanmajchrak milanmajchrak changed the title Transaction bug - see differencies (#845) Transaction bug - close context in finally block (#845) Jan 16, 2025
Copy link

@vidiecan vidiecan left a comment

Choose a reason for hiding this comment

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

add info to context that a the transaction is alive in the dbconnection BUT not for the ctor that set dbconnect in its params

Copy link

@vidiecan vidiecan left a comment

Choose a reason for hiding this comment

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

see comment

@milanmajchrak milanmajchrak merged commit dc57ffd into dtq-dev Jan 16, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants