Skip to content
This repository has been archived by the owner on Jan 25, 2021. It is now read-only.

Uploads slower (v.2.2 beta) #611

Closed
iksi opened this issue Nov 3, 2015 · 4 comments
Closed

Uploads slower (v.2.2 beta) #611

iksi opened this issue Nov 3, 2015 · 4 comments
Milestone

Comments

@iksi
Copy link
Contributor

iksi commented Nov 3, 2015

When there are a lot of files for the page (I know this is generally not a good idea to have a lot of them) there is a significant speed difference between the 2.1.1 and the 2.2 panel. For instance adding a file to a page that already has a 100 files happens fast on panel 2.1.1 but takes a very long time on panel 2.2 (looking at the spinner for around 15 seconds on localhost). The extra file is small (100kb). As I said it’s generally not a good idea to have a lot of files for a page, but I couldn’t help noticing this speed difference.

Update: it seems to be that it’s not the upload itself that’s taking long. The file appears quickly in the filesystem but the panel keeps showing the spinner.

@lukasbestle lukasbestle added this to the 2.2 milestone Nov 3, 2015
@distantnative
Copy link

Can't reproduce that. Put about 110 pdf files in a page and when I add another pdf file up to 100kb, it might not be lightning fast, but within 2-3 seconds at most.

Could you tell me what files that are or if you have any files settings in your blueprint? Cause I'd assume that it's rather a PHP error somewhere on the way, even after the upload as it looks like.

@iksi
Copy link
Contributor Author

iksi commented Nov 9, 2015

They were jpegs. The site contains a lot of images, and the amount of files in the /thumbs folder went way over 10,000. I currently can’t reproduce it myself as I changed the way thumbs are handled: I’ve added a thumbs.destination function so that thumbnails are stored with the content itself. Files settings were as follows:

files:
  fields:
    video:
      label: Video
      type: url
  sortable: true

@bastianallgeier
Copy link
Contributor

We should keep an eye on it, but close it for now. I guess it really had something to do with the thumbnails. I'm working on a better solution for that anyway.

@iksi
Copy link
Contributor Author

iksi commented Nov 9, 2015

👍🏼

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants