Login | Register
My pages Projects Community openCollabNet

Discussions > dev [DISABLED] > Re: [patch] revprop packing for fsfs

subversion
Discussion topic

Back to topic list

Re: [patch] revprop packing for fsfs

Author hwright
Full name Hyrum K. Wright
Date 2009-05-21 12:05:55 PDT
Message On May 20, 2009, at 2:40 AM, webpost at tigris dot org wrote:

> I won't criticise SQLite, but if there was a case for keeping the
> flat file system wouldn't a simple solution be to write modified
> revprops as a single file, and mark the revprop in the packed file
> as obsolete?
>
> Then reading a revprop reads the packed file as normal, but if the
> flag is set, read the revprop file that will be found on disk.
>
> A regular pack operation will then recombine the revprop into the
> packed file.
>
> Just my 2p, but if SQLite gives more functionality or is just faster
> - then I'm well happy with that!

Still haven't had time to review the patch, though I have time to do
some drive-by kibitzing. :)

Disks have improved vastly since Subversion debuted several years
ago. When it first came out, we spent a lot of time optimizing disk
accesses, on both the client and server. (Incidentally, this led to a
lot of mess in the working copy library, precipitating the current
rewrite known as wc-ng.)

The increase in disk performance means that we can sacrifice a little
bit of performance, for space and maintainability considerations.
Subversion should be in the business of doing version control, not
worrying about low-level file storage formats. Using a separate
library to manage the on-disk formats let's us concentrate on solving
version control problems. There isn't anything special about sqlite
per se, it's just the most convenient choice for the time being.

That's probably more than you'd like to hear, it's something I've been
thinking about recently, so it all spilled out. (And yes, Paul, I'm
still planning on reviewing the revprop-sqlite patch. Where did I put
those tuits?)

-Hyrum

« Previous message in topic | 11 of 12 | Next message in topic »

Messages

Show all messages in topic

revprop packing for fsfs chipig Paul Querna 2009-04-27 18:12:52 PDT
     Re: revprop packing for fsfs peters Peter Samuelson 2009-04-28 14:49:41 PDT
         Re: revprop packing for fsfs Daniel Shahaf <d dot s at daniel dot shahaf dot name> Daniel Shahaf <d dot s at daniel dot shahaf dot name> 2009-04-28 23:58:45 PDT
             Re: revprop packing for fsfs chipig Paul Querna 2009-04-29 15:19:04 PDT
                 Re: revprop packing for fsfs hwright Hyrum K. Wright 2009-04-29 16:33:42 PDT
                     [patch] revprop packing for fsfs chipig Paul Querna 2009-04-30 23:55:32 PDT
                         Re: [patch] revprop packing for fsfs Daniel Shahaf <d dot s at daniel dot shahaf dot name> Daniel Shahaf <d dot s at daniel dot shahaf dot name> 2009-05-01 03:14:44 PDT
                         Re: [patch] revprop packing for fsfs chipig Paul Querna 2009-05-07 10:02:10 PDT
                             Re: [patch] revprop packing for fsfs cmpilato C. Michael Pilato 2009-05-07 10:09:02 PDT
                                 RE: Re: [patch] revprop packing for fsfs webpost at tigris dot org webpost at tigris dot org 2009-05-20 02:40:34 PDT
                                     Re: [patch] revprop packing for fsfs hwright Hyrum K. Wright 2009-05-21 12:05:55 PDT
                                         RE: Re: [patch] revprop packing for fsfs gbjbaanb Andy Bolstridge 2009-05-22 00:18:33 PDT
Messages per page: