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

Release page loads slow with a lot of tags #490

Closed
6 tasks
deanpcmad opened this issue Dec 26, 2016 · 13 comments · Fixed by #527
Closed
6 tasks

Release page loads slow with a lot of tags #490

deanpcmad opened this issue Dec 26, 2016 · 13 comments · Fixed by #527
Assignees
Labels
Milestone

Comments

@deanpcmad
Copy link

deanpcmad commented Dec 26, 2016

Copied from gogs/gogs#3094

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

Description

I've just created a mirror repository for the rails repo and upon viewing the "Releases" tab, it takes forever to load, sometimes not loading at all which can cause nginx to timeout

Please see #491 as well

@lunny lunny added this to the 1.1.0 milestone Dec 27, 2016
@lunny lunny added the type/enhancement An improvement of existing functionality label Dec 27, 2016
@bkcsoft bkcsoft added type/bug and removed type/enhancement An improvement of existing functionality labels Dec 28, 2016
@bkcsoft
Copy link
Member

bkcsoft commented Dec 28, 2016

not an enhancement, it's a bug. I added release pagniation a while back but I seem to have missed this for-loop here that still loop over all tags 😒 https://github.com/go-gitea/gitea/blob/master/routers/repo/release.go#L77

@bkcsoft
Copy link
Member

bkcsoft commented Dec 28, 2016

Just flipping that for-loop around and using ctx.Repo.GitRepo.GetTag(r.TagName) should make it way faster, and TBH we probably don't need the caching anymore since we're not fetching all tags in one go :)

@deanpcmad
Copy link
Author

Yeah I saw there is pagination so I did wonder why it was slow, that explains it :)

@JetpackDuba
Copy link

Will you fix it for 1.0.1 or only 1.1? Imo it should be in both :P

@lunny lunny self-assigned this Dec 29, 2016
@lunny
Copy link
Member

lunny commented Dec 31, 2016

@deanpcmad, could you please test if #527 resolved your problem since I think there are also some improvements should be taken in future.

@deanpcmad
Copy link
Author

@lunny sure, is there a build for it?

@lunny
Copy link
Member

lunny commented Dec 31, 2016

https://dl.gitea.io/gitea/master/, you can find your proper version.

@JetpackDuba
Copy link

I'm not who opened the issue but in my case using Windows 10 and MSSQL, I've cloned VS CODE github repo and tags page takes 8.8 seconds to load :S

@deanpcmad
Copy link
Author

Hey, I've just tried it (version 09dabe2) but it is still very slow to load the tags page. If you want to test it yourself locally, mirror the https://github.com/rails/rails repo

@JetpackDuba
Copy link

Now it loads in 1sec! That's better!

@lunny
Copy link
Member

lunny commented Jan 8, 2017

Yes. But it's not perfect. It needs further improvement later.

@lunny
Copy link
Member

lunny commented Jan 14, 2017

On macOS and sqlite, rails still spent 10s.

@lunny
Copy link
Member

lunny commented Nov 1, 2017

https://try.gitea.io/lunny/rails/releases. v1.2+ have resolve this issue entirely.

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

Successfully merging a pull request may close this issue.

4 participants