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

Compare changes and new pull requests take a very long time. #2837

Closed
2 of 7 tasks
MsBuggy opened this issue Nov 3, 2017 · 18 comments · Fixed by #7797
Closed
2 of 7 tasks

Compare changes and new pull requests take a very long time. #2837

MsBuggy opened this issue Nov 3, 2017 · 18 comments · Fixed by #7797

Comments

@MsBuggy
Copy link

MsBuggy commented Nov 3, 2017

  • Gitea version (or commit ref): 1.2 windows-amd64
  • Git version: 2.11.0 windows
  • Operating system: windows 2008 R2 16 CPU cores and 16GB RAM
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant
  • Log gist:

Description

I try to create pull requests and the 'compare changes' step takes 10 minutes! Create pull requests make the browser crashed!

@lflare
Copy link
Contributor

lflare commented Nov 3, 2017

How big of a repository are we talking about here? Do you have any logs?

@webjoel
Copy link
Contributor

webjoel commented May 15, 2018

In my case is a little slow too, repo is very big, an solution is add gauge/loading image for feedback to user. It's possible?

If yes, is good add too in apply merge pull request, what is very slow too e no exixsts feedback to user, only the loading in the browser title.

My repo size is: 1,8 GB.

Issue related: #601

@adelowo
Copy link
Member

adelowo commented Dec 2, 2018

I am currently experiencing this too on a project with a size of 146MB. :)

@lunny
Copy link
Member

lunny commented Dec 2, 2018

@adelowo maybe you can send a pull request.

@adelowo
Copy link
Member

adelowo commented Dec 2, 2018

@lunny I will try to take a look at it in the coming week hopefully.

By the way, mine takes 3 -5 minutes without causing a browser crash

@stale
Copy link

stale bot commented Jan 31, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale stale bot added the issue/stale label Jan 31, 2019
@lunny
Copy link
Member

lunny commented Feb 6, 2019

@MsBuggy @webjoel @adelowo is this still a problem?

@webjoel
Copy link
Contributor

webjoel commented Feb 6, 2019

@lunny I am using an old version (commit 1b2aff0), I have not yet been able to upgrade to the latest version, so I'm not sure if the problem persists in the new ones, but in this one I'm using the slowness continues.

@adelowo
Copy link
Member

adelowo commented Feb 6, 2019

@lunny I no longer have access to the repository so I will not be able to test that

@zeripath
Copy link
Contributor

zeripath commented Feb 6, 2019

Ok that Gitea version is before the substantial speed up pr #4921 @typeless made was pulled.

@webjoel
Copy link
Contributor

webjoel commented Feb 6, 2019

@lunny @lafriks Is it possible for this PR ( #4921 ) to inner in 1.7.x release? Or is it only 1.8?

Anyway, for me can close this issue, since other related issues were also closed, so the best should have generated performance gain.

@lafriks
Copy link
Member

lafriks commented Feb 6, 2019

@webjoel only bug fixes are backported

@lafriks lafriks closed this as completed Feb 6, 2019
@lunny lunny removed the issue/stale label Feb 7, 2019
@1902sysad
Copy link

im having this issue, is there any workarounds?

@typeless
Copy link
Contributor

typeless commented Aug 8, 2019

I encountered this a couple of times. I tried to sample the timings with ps -u git -o cmd manually and got

CWD CMD
-   /volume1/homes/git/gitea/gitea web
-   /bin/git -c credential.helper= -c protocol.version=2 -c filter.lfs.required= -c filter.lfs.smudge= -c filter.lfs.clean= remote add -f 1565248623376910366 /volume1/homes/git/gitea/repositories/saturn/large-repo.git
-   /var/packages/Git/target/libexec/git-core/git fetch 1565248623376910366
-   git-upload-pack /volume1/homes/git/gitea/repositories/saturn/large-repo.git
-   /var/packages/Git/target/libexec/git-core/git rev-list --objects --stdin --not --all --quiet

I guess git remote add -f is probably the one taking long time.

@typeless
Copy link
Contributor

typeless commented Aug 8, 2019

Looking into it further, that should be at

if err = repo.AddRemote(tmpRemote, basePath, true); err != nil {

@typeless
Copy link
Contributor

typeless commented Aug 8, 2019

A quick workaround, I assume, is to decouple git remote add and git fetch.
And then make git fetch fetch only the branch in question, not all the other refs.

@typeless
Copy link
Contributor

typeless commented Aug 8, 2019

@1902sysad Hey, if you know how to build Gitea from source, please try https://github.com/go-gitea/gitea/pull/7797/files to see if it fixes your problem.

@1902sysad
Copy link

Thank you we will be trying this soon

@lunny lunny reopened this Aug 8, 2019
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants