Modify

Ticket #3 (closed defect: fixed)

Opened 7 years ago

Last modified 6 years ago

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:1 Changed 7 years ago by and

  • Milestone set to 0.2

comment:2 Changed 7 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 7 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 7 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 7 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 7 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 7 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

WorkflowException?

Error Value

Could not get info: review_history

comment:9 Changed 7 years ago by andycat

comment:10 Changed 7 years ago by andycat

comment:12 Changed 7 years ago by andycat

  • Status changed from new to closed
  • Resolution set to fixed

comment:13 Changed 7 years ago by low cost health insurance

  • Status changed from closed to reopened
  • Resolution fixed deleted

comment:17 Changed 7 years ago by andycat

  • Status changed from reopened to closed
  • Resolution set to fixed
View

Add a comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.