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
Make code sequential and remove deadlock #3210
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on @srfrog)
ee/backup/s3_handler.go, line 216 at r1 (raw file):
readManifests := func(objects []string) (map[int]*Manifest, error) { res := make(map[int]*Manifest)
We don't need to read all the manifests. Just the ones until the first full backup.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on @manishrjain)
ee/backup/s3_handler.go, line 216 at r1 (raw file):
Previously, manishrjain (Manish R Jain) wrote…
We don't need to read all the manifests. Just the ones until the first full backup.
We read the manifest to know which files to restore.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewable status: 0 of 3 files reviewed, 1 unresolved discussion (waiting on @manishrjain)
* make code sequential and remove deadlock * simplify code further * worker/backup_ee.go: unwarned change that broke tests after merge * ee/backup/docker-compose.yml: add user to docker compose file to fix testing in Linux * ee/backup/docker-compose.yml: add --cwd to docker service command to fix dir ownership * ee/backup/docker-compose.yml: remove working_dir it interferes with Linux testing
This change is