[mythtv-commits] Ticket #2708: patch: Allow remote FE/BE to prefer to use the myth protocol

[MythTV]

#2708: patch: Allow remote FE/BE to prefer to use the myth protocol
--------------------------------------+-------------------------------------
 Reporter:  gdragon at jetcom dt org  |       Owner:  ijr    
     Type:  patch                     |      Status:  new    
 Priority:  minor                     |   Milestone:  unknown
Component:  mythtv                    |     Version:  0.20   
 Severity:  medium                    |  
--------------------------------------+-------------------------------------
 On slave FE's it is generally recommended by various posts in the mailing
 lists that the myth protocol is better suited to handle playback of
 streams rather than use a NFS mounted directory. This is especially true
 when playing back HD streams. The recommendation here is to not have the
 recording directory mounted and it will use the protocol.

 On a setup with a slave FE & BE that writes to the master common recording
 storage, this is not feasible as we need write access to the recording
 directory. In this case we need to write to the nfs mount, but prefer the
 mythprotocol when playingback. This is especially true in a diskless
 backend, where there is no local disk.

 This patch modifies GetPlaybackURL to return a myth:// url rather than a
 local file handle.[[BR]]
 - It defaults to a local file if the recording was done on the host.[[BR]]

 - A hidden database configuration item is added called
 "PreferMythProtocol". This item is applicable to each slave FE that would
 require this functionality.  I wouldn't expect that you'd want to set this
 on the master server.  I did not make this configurable on the GUI, as I'm
 not sure where it'd go(I can add this if desired). [[BR]]

 - There is an added argument to the function to indicate that the caller
 request read/write access to the file. In this case the file handle is
 used if available. This parameter defaults to off, aka read-only.[[BR]]

 - I also removed one argument to the function. It was an unused defaulted
 parameter, that didn't seem to have any value. It would have meant 2
 defaulted parameters if left in. If deemed important, I'll revise the
 patch to add it back.

 I've tested this on my setup, one master, and 2 slave machines. Both
 slaves are diskless, one has a tuner, and records SD from a cable box. The
 patch was developed & tested on the fixes branch, but applies cleanly
 (with minor offset) to svn head.

 This patch makes a big difference on my setup, I can now watch HD streams
 cleanly on slave boxes, while backend is recording 2 hd streams and remote
 slave is recording.

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/2708>
MythTV <http://www.mythtv.org/>
MythTV