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

[BTS-664] Hotbackup transfer job cleanup part 2 #15275

Merged
merged 3 commits into from
Dec 6, 2021

Conversation

neunhoef
Copy link
Member

@neunhoef neunhoef commented Dec 3, 2021

Even more cleanups for hotbackup transfer jobs.

It was found that the previous round of cleanups did not consider
old, unfinished, seemingly ongoing jobs and transfer locks.

This PR adds cleanup for all those to keep the agency size under
control.

This PR is accompanied by an enterprise repo PR: https://github.com/arangodb/enterprise/pull/826

Scope & Purpose

(Please describe the changes in this PR for reviewers - mandatory)

  • [*] 💩 Bugfix (requires CHANGELOG entry)
  • [*] 📖 CHANGELOG entry made

Backports:

Related Information

(Please reference tickets / specification / other PRs etc)

Testing & Verification

(Please pick either of the following options)

  • [*] The behavior in this PR was manually tested
  • [*] This change is already covered by existing tests, such as (please describe tests).
  • [] This PR adds tests that were used to verify all changes:
    - [
    ] Added new C++ Unit tests

@neunhoef neunhoef requested a review from a team as a code owner December 3, 2021 14:08
@neunhoef neunhoef added this to the devel milestone Dec 3, 2021
@neunhoef neunhoef self-assigned this Dec 3, 2021
@neunhoef neunhoef requested a review from kvahed December 3, 2021 14:08
Copy link
Contributor

@kvahed kvahed left a comment

Choose a reason for hiding this comment

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

LGTM

@neunhoef neunhoef merged commit 40db452 into devel Dec 6, 2021
@neunhoef neunhoef deleted the bug-fix/bts-664-hotbackup-transfer-cleanup2 branch December 6, 2021 13:32
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