Ticket #3 (closed defect: fixed)
Add filesize data into the catalog index
| Reported by: | and | Owned by: | andycat |
|---|---|---|---|
| Priority: | blocker | Milestone: | 0.2 |
| Component: | component1 | Severity: | |
| Keywords: | Cc: | ||
| Who will test this: |
Description
The video view page displays file size of a video. Currently this wakes up the object from the zodb - ie we are going beyond the "brain" object, which is the catalog's object - cached info. This means the ATVideo obj is loaded from the zodb - one attribute of which is the ExternalStorage? object pointing to the actual file on the filesystem. This system makes page loading for large videos very long and sometimes has the effect of the plone instance running out of memory.
We dont need to do this, but we just need to add the filesize data into the catalog indexing system - so it is in the brain object (ie the cache).
Attachments
Change History
comment:2 Changed 6 years ago by and
This issue just occured on the EM server as Sam was editing a large video. this was the error
ERROR The requested URL could not be retrieved
While trying to retrieve the URL: http://127.0.0.1:8080/VirtualHostBase/http/www.engagemedia.org:80/Beta/VirtualHostRoot/
The following error was encountered:
- Connection to 127.0.0.1 Failed
The system returned:
(111) Connection refused
The remote host or network may be down. Please try the request again.
Your cache administrator is contact@…. Generated Mon, 16 Jul 2007 16:10:34 GMT by www.engagemedia.org (squid/2.6.STABLE5)
comment:3 Changed 6 years ago by anonymous
We should investigate using BlobField? in the schema inside ATVideo http://dev.plone.org/plone/ticket/6805
Need to use ZODB >= 3.8 (that is, you need Zope >= 2.11.x )
comment:4 Changed 6 years ago by andycat
I have removed video object lookups from index_html in ATVideo This avoids going to the catalog to get the object, and instead just uses the brain object from the catalog search results. I add a new index inside the ATVideo schema , so the template can still check whether an thumbnail exists or not. http://dev.plone.org/collective/changeset/48349
XXX Missing formattedDuration on the index_html template, since its an object method.
I have added a class attribute to cache the file size, when used in video_view.pt and else where http://dev.plone.org/collective/changeset/48355
comment:6 Changed 6 years ago by andycat
The following templates could do with having getObject removed, to avoid asking the catalog to get the item:
author.cpt videofolder_listing.cpt folder_listing.cpt
comment:7 Changed 6 years ago by anonymous
I have added in DurationOfVideo? as a index/metadata on the catalog http://dev.plone.org/collective/changeset/48662 http://dev.plone.org/collective/changeset/48664
and put it back into index_html http://dev.plone.org/collective/changeset/48670
We are good to go , if alls well on testing
comment:8 Changed 6 years ago by and
look good except that is says "video duration" before saying the duration. I think it will be pretty obvious to most people that the minutes/seconds refers to the duration.
Also I can't get to any of the videos in testing.engagemedia to test any further. I get
Error Type
Error Value
Could not get info: review_history
comment:9 Changed 6 years ago by andycat
Author changeset - http://dev.plone.org/collective/changeset/49369
comment:10 Changed 6 years ago by andycat
Videofolder changeset - http://dev.plone.org/collective/changeset/49371
comment:11 Changed 6 years ago by andycat
comment:12 Changed 6 years ago by andycat
- Status changed from new to closed
- Resolution set to fixed
more silly bugfixes http://dev.plone.org/collective/changeset/49388 http://dev.plone.org/collective/changeset/49389 http://dev.plone.org/collective/changeset/49390 http://www.plumi.org/changeset/210
all these changes have gone into stable.
comment:13 Changed 5 years ago by low cost health insurance
- Status changed from closed to reopened
- Resolution fixed deleted
comment:17 Changed 5 years ago by andycat
- Status changed from reopened to closed
- Resolution set to fixed
