Skip to content
This repository was archived by the owner on Sep 11, 2020. It is now read-only.

Unable to push when there are a lot of merged merge requests history in ref #530

Closed
cindy52 opened this issue Aug 1, 2017 · 3 comments
Closed

Comments

@cindy52
Copy link

cindy52 commented Aug 1, 2017

Func hashListToSet got stuck when there are a lots of values in hashes list
func hashListToSet(hashes []plumbing.Hash) map[plumbing.Hash]bool {
result := make(map[plumbing.Hash]bool)
for _, hash := range hashes {
result[hash] = true
}
return result
}

@cindy52 cindy52 changed the title Unable to push when there are a lot of merged merge requests in ref Unable to push when there are a lot of merged merge requests history in ref Aug 1, 2017
@orirawlings
Copy link
Contributor

Hi @cindy52,

Do you have more information to help us reproduce the issue? Maybe an example git repository that exhibits the problem and some go code that triggers the problem in the go-git library? Perhaps some output that you used to help narrow down the problem to the hashListToSet function in particular?

@cindy52
Copy link
Author

cindy52 commented Aug 1, 2017

I try to push to a repository that has 1000+ merged merge requests simply using following code:
err = r.Push(&git.PushOptions{Auth: auth}), then it got stuck at hashListToSet and the result of debugger saying that connected to loacalhost without no other variables. But it works fine with other repositories with little merged request history.

@cindy52
Copy link
Author

cindy52 commented Aug 2, 2017

It's been fixed here revlist: ignore all objects reachable from ignored objects #516 I will close the issue. Sorry for confusion.

@cindy52 cindy52 closed this as completed Aug 2, 2017
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants