[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Feature request: svnadmin load shouldn't load existing revisions

From: Ulrich Eckhardt <eckhardt_at_satorlaser.com>
Date: Tue, 28 Sep 2010 09:19:36 +0200

Hi!

I just made a stupid mistake that cost me a few hours for loading a multi-GiB
repository. What I did was to load a repository from a backup dumpfile, which
went to revision 37344 (and took several hours). I then proceeded to create
an incremental dumpfile from the life repository, and made an off-by-one
mistake there:

  svnadmin dump --incremental -r 37344:HEAD ...

I then copied the dumpfile to a different machine and loaded it there, but
after the first revision SVN told me that it "Committed new rev 37345 (loaded
from original rev 37344)" which is exactly _not_ what I wanted, because now
the new repository is unusable with existing working copies because its
revisions are one off.

Two things where I think svnadmin could do better:
1. The dump was incremental
I would expect that this requires some checking if the "patch" can actually be
applied. Maybe it's just my bad luck that there were no conflicts that caused
loading the dumpfile to fail.

2. The revision numbers don't match
My call basically means that I'm telling svnadmin to create a revision that
already exists. Instead of renumbering the revision, bailing out with an
error would have been a good idea. Of course, there are legitimate cases
where this is needed, so a commandline switch would be needed to override
this check, but by default I would refuse loading the dumpfile. Maybe also
skipping existing revisions would be an option, at least I would have found
that helpful.

Thanks!

Uli

-- 
ML: http://subversion.apache.org/docs/community-guide/mailing-lists.html
FAQ: http://subversion.apache.org/faq.html
Docs: http://svnbook.red-bean.com/
**************************************************************************************
Sator Laser GmbH, Fangdieckstraße 75a, 22547 Hamburg, Deutschland
Geschäftsführer: Thorsten Föcking, Amtsgericht Hamburg HR B62 932
**************************************************************************************
           Visit our website at <http://www.satorlaser.de/>
**************************************************************************************
Diese E-Mail einschließlich sämtlicher Anhänge ist nur für den Adressaten bestimmt und kann vertrauliche Informationen enthalten. Bitte benachrichtigen Sie den Absender umgehend, falls Sie nicht der beabsichtigte Empfänger sein sollten. Die E-Mail ist in diesem Fall zu löschen und darf weder gelesen, weitergeleitet, veröffentlicht oder anderweitig benutzt werden.
E-Mails können durch Dritte gelesen werden und Viren sowie nichtautorisierte Änderungen enthalten. Sator Laser GmbH ist für diese Folgen nicht verantwortlich.
**************************************************************************************
Received on 2010-09-28 09:20:19 CEST

This is an archived mail posted to the Subversion Users mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.