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

fix(tasks): frozen tasks with pgpool #2264

Merged
merged 4 commits into from
Dec 6, 2024

Conversation

sylvlecl
Copy link
Member

@sylvlecl sylvlecl commented Dec 6, 2024

Backport of #2263 on hotfix v2.18.2

Signed-off-by: Sylvain Leclerc <[email protected]>
Signed-off-by: Sylvain Leclerc <[email protected]>
Signed-off-by: Sylvain Leclerc <[email protected]>
@sylvlecl sylvlecl changed the base branch from dev to hotfix/v2.18.2 December 6, 2024 15:39
@sylvlecl sylvlecl changed the title fix frozen tasks with pgpool fix(tasks): frozen tasks with pgpool Dec 6, 2024
@sylvlecl sylvlecl added this to the v2.18.2 milestone Dec 6, 2024
@sylvlecl sylvlecl merged commit dea1b75 into hotfix/v2.18.2 Dec 6, 2024
11 of 13 checks passed
@sylvlecl sylvlecl deleted the fix/fix-frozen-tasks-pgpool branch December 6, 2024 16:35
skamril pushed a commit that referenced this pull request Dec 11, 2024
This work aims at fixing issues encountered with load balanced pgpool,
where a commit is not always instantly visible in another session (when
read from replica).

Fixes are:
- restore a workaround which consists in re-trying the read until success
- ensure exceptions occurring in the task execution thread are
  ALL caught

Also removing a technical debt:
- remove optional session from task job repo which was just used in tests.
  It did not work with code which uses both the repo and also the session
  from the singleton db.session.

Signed-off-by: Sylvain Leclerc <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant