Pull request view take long time to load #7420

Closed
opened 2025-11-02 07:25:24 -06:00 by GiteaMirror · 9 comments
Owner

Originally created by @zuhairamahdi on GitHub (Jun 2, 2021).

  • Gitea version (or commit ref): 1.14.2
  • Git version:
  • Operating system: Windows
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
  • Log gist:

Description

we are starting to notice that the pull request pages (especially if it is a big one 'many commits') take a long time to load. and the files changed section even take longer.

note that due to the long time it takes to open the 'Files Changed' I was not able to take screenshot (we are talking about minutes not seconds)

Screenshots

image
image

Originally created by @zuhairamahdi on GitHub (Jun 2, 2021). <!-- NOTE: If your issue is a security concern, please send an email to security@gitea.io instead of opening a public issue --> - Gitea version (or commit ref): 1.14.2 - Git version: - Operating system: Windows <!-- Please include information on whether you built gitea yourself, used one of our downloads or are using some other package --> <!-- Please also tell us how you are running gitea, e.g. if it is being run from docker, a command-line, systemd etc. ---> <!-- If you are using a package or systemd tell us what distribution you are using --> - Database (use `[x]`): - [ ] PostgreSQL - [ ] MySQL - [x] MSSQL - [ ] SQLite - Can you reproduce the bug at https://try.gitea.io: - [ ] Yes (provide example URL) - [ ] No - Log gist: <!-- It really is important to provide pertinent logs --> <!-- Please read https://docs.gitea.io/en-us/logging-configuration/#debugging-problems --> <!-- In addition, if your problem relates to git commands set `RUN_MODE=dev` at the top of app.ini --> ## Description we are starting to notice that the pull request pages (especially if it is a big one 'many commits') take a long time to load. and the files changed section even take longer. note that due to the long time it takes to open the 'Files Changed' I was not able to take screenshot (we are talking about minutes not seconds) ## Screenshots ![image](https://user-images.githubusercontent.com/6876269/120512341-f15f4300-c3d3-11eb-8e77-8607fd15afbb.png) ![image](https://user-images.githubusercontent.com/6876269/120512583-2cfa0d00-c3d4-11eb-9ed9-421ba6e6eaaf.png)
GiteaMirror added the performance/speedissue/workaround labels 2025-11-02 07:25:24 -06:00
Author
Owner

@zuhairamahdi commented on GitHub (Jun 2, 2021):

this is how much it take to load 'Files Changed' for me:
image

@zuhairamahdi commented on GitHub (Jun 2, 2021): this is how much it take to load 'Files Changed' for me: ![image](https://user-images.githubusercontent.com/6876269/120513292-ca554100-c3d4-11eb-8ae3-e483a6c8a657.png)
Author
Owner

@zuhairamahdi commented on GitHub (Jun 2, 2021):

even when viewing a single commit changes it is taking a long time:
image
this specific commit had only 2 lines changed

@zuhairamahdi commented on GitHub (Jun 2, 2021): even when viewing a single commit changes it is taking a long time: ![image](https://user-images.githubusercontent.com/6876269/120515167-bb6f8e00-c3d6-11eb-9378-30db9580731c.png) this specific commit had only 2 lines changed
Author
Owner

@lunny commented on GitHub (Jun 2, 2021):

Have you tested gogit version?

@lunny commented on GitHub (Jun 2, 2021): Have you tested gogit version?
Author
Owner

@zuhairamahdi commented on GitHub (Jun 2, 2021):

@lunny how can I test that?

@zuhairamahdi commented on GitHub (Jun 2, 2021): @lunny how can I test that?
Author
Owner

@zuhairamahdi commented on GitHub (Jun 3, 2021):

Hi @lunny we use the executable for windows and I'm not sure how to test the version of gogit. this is giving us a very hard time to do code reviews and merge pull requests.

@zuhairamahdi commented on GitHub (Jun 3, 2021): Hi @lunny we use the executable for windows and I'm not sure how to test the version of gogit. this is giving us a very hard time to do code reviews and merge pull requests.
Author
Owner

@lunny commented on GitHub (Jun 3, 2021):

@zuhairamahdi You can find the download link from https://dl.gitea.io/gitea/1.14.2/gitea-1.14.2-gogit-windows-4.0-amd64.exe , for windows and v1.14.2, we generated two versions release.

@lunny commented on GitHub (Jun 3, 2021): @zuhairamahdi You can find the download link from https://dl.gitea.io/gitea/1.14.2/gitea-1.14.2-gogit-windows-4.0-amd64.exe , for windows and v1.14.2, we generated two versions release.
Author
Owner

@zuhairamahdi commented on GitHub (Jun 3, 2021):

Hi @lunny once we changed to the gogit version there was a considerable performance boost:

  • Pull request page:
    image
  • Files changed views:
    image
@zuhairamahdi commented on GitHub (Jun 3, 2021): Hi @lunny once we changed to the gogit version there was a considerable performance boost: - Pull request page: ![image](https://user-images.githubusercontent.com/6876269/120660519-12856980-c490-11eb-8af0-b458d75d4572.png) - Files changed views: ![image](https://user-images.githubusercontent.com/6876269/120660631-32b52880-c490-11eb-8408-cd9833b8df8e.png)
Author
Owner

@lunny commented on GitHub (Jun 4, 2021):

So this is a known issue that we are trying to resolve, the problem of gogit version is that may spend more time than git version.

@lunny commented on GitHub (Jun 4, 2021): So this is a known issue that we are trying to resolve, the problem of gogit version is that may spend more time than git version.
Author
Owner

@wxiaoguang commented on GitHub (Apr 16, 2022):

Caused by Git for Windows

@wxiaoguang commented on GitHub (Apr 16, 2022): Caused by Git for Windows
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#7420