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

Run merging of layers in parallel #64

Open
goldmann opened this issue Mar 23, 2016 · 1 comment
Open

Run merging of layers in parallel #64

goldmann opened this issue Mar 23, 2016 · 1 comment

Comments

@goldmann
Copy link
Owner

If it is possible to run merging of layers in parallel - we should consider this.

@goldmann goldmann added this to the Future milestone Mar 23, 2016
@goldmann
Copy link
Owner Author

After skimming the code - it may be harder to do than I thought. While merging layers we do check for each layer we want to merge if the squashed layer already contains specific files. This means that we would need to synchronize on this call. But even if we synchronize, some other layer could add the same file in the meantime. It seems that parallel merging is not an easy improvement and it's even unclear if this is doable.

@goldmann goldmann removed the squash label Jul 25, 2016
@goldmann goldmann removed this from the Future milestone Mar 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant