-
Notifications
You must be signed in to change notification settings - Fork 66
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
Precompute list of needed blobs before a checkout #36
Comments
For discussion: To get this ready for the Demo milestone, we may want an in-between:
Long-term, we definitely want Git to handle this by itself. However, to get this full list precomputed we will need Git changes for the precompute (this issue) and the batched read-object hook (#7). |
@derrickstolee Thanks for starting the discussion. I think your proposal satisfies the requirements for manipulating the set of folders in the sparse enlistment without moving what HEAD points to, but I don't think it fully addresses computing the list of blobs when we checkout or reset. Right now |
True. My proposal will only fix that initial hydration on a fresh clone. A branch-switch would still use the read-object hook for the blobs that changed between. |
Labeling this issue as stale. There has been no activity for 30 days. Remove stale label or comment or this issue will be closed in 7 days. |
Labeling this issue as stale. There has been no activity for 30 days. Remove stale label or comment or this issue will be closed in 7 days. |
This is being addressed in: microsoft/git#191 |
marking closed along with #7 |
…ject hook See microsoft/git#191 for the Git code regarding the GVFS transport layer. This can work in tandem with the read-object hook, but it _should_ make the read-object hook irrelevant. * Delete all references to the read-object hook in the product code. This unblocks #4 and the [Mount Removal] tasks #15, #132, #133, #135, #136, and possible others. * Delete the Sparse Verb in favor of `git sparse-checkout set`. * Delete the `BlobPrefetcher` and all references to `--files` or `--folders` prefetching. Resolves #6, #7, #36.
Rather than fall back on read-object one-by-one, let's precompute what's needed, a la partial.
The text was updated successfully, but these errors were encountered: