#5895: [with patch; needs work] Limit number of worksheet snapshots
----------------------+-----------------------------------------------------
 Reporter:  rbeezer   |       Owner:  rbeezer   
     Type:  defect    |      Status:  new       
 Priority:  major     |   Milestone:  sage-3.4.2
Component:  notebook  |    Keywords:            
----------------------+-----------------------------------------------------

Comment(by rbeezer):

 Replying to [comment:2 was]:
 > It reads better, will work, *and* will avoid subtle race conditions.

 Yes, that was basically my first version.

 >The way you have stuff setup in this patch, if there is ever a situation
 where two snapshots are made, but this function isn't called (e.g., due to
 some weird race conditions), then one goes back to having potentially
 thousands of snapshots.

 Or if the value of {{{max_snaps}}} is configurable and is decreased at
 some future time.

 > Also, I see no reason to not delete snapshots from old worksheets too.

 I guess I worry that it is not very courteous to trash anything in
 someone's home directory without warning.  ;-)  Other than a note on sage-
 devel with the announcement of a new version, are there any good ways to
 put out the word to more "casual" users?

 > In fact, I very much hope that when I apply this patch, then directories
 with tons of snapshots on sagenb.org and my laptop will have their
 excessive snapshots deleted, at least if the corresponding worksheets are
 used.

 I believe that will be the case.  On a personal setup, on shutting down a
 server, it seems *every* worksheet is examined for a save, but I think
 there would need to be an edit of the worksheet to get to this new
 routine.  If this routine was called sooner in {{{save_worksheet()}}} then
 there might be a wholesale purge the first time the server is shut down
 (though in the long-run the routine would be called many times when it was
 unnecessary).

 Thanks for the comments - I'll be back to it tomorrow.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5895#comment:4>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of 
Reinventing the Wheel

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to