Re: svn commit: r1700799 - in /subversion/trunk/subversion: include/svn_io.h libsvn_subr/stream.c svnadmin/svnadmin.c svnfsfs/load-index-cmd.c tests/libsvn_subr/stream-test.c

2015-09-14 Thread Stefan Fuhrmann
On Sat, Sep 12, 2015 at 1:46 AM, Evgeny Kotkov wrote: > Julian Foad writes: > > > Also for: > >svnrdump load > >svnmucc put - > > > > ? (These just came to mind when I thought about it.) > > I think, yes. Thank you for pointing

Re: svn commit: r1700799 - in /subversion/trunk/subversion: include/svn_io.h libsvn_subr/stream.c svnadmin/svnadmin.c svnfsfs/load-index-cmd.c tests/libsvn_subr/stream-test.c

2015-09-14 Thread Evgeny Kotkov
Stefan Fuhrmann writes: >> Julian Foad writes: >> >> > Also for: >> >svnrdump load >> >svnmucc put - >> > >> > ? (These just came to mind when I thought about it.) >> >> I think, yes. Thank you for pointing this out, Julian. >

Re: svn commit: r1700799 - in /subversion/trunk/subversion: include/svn_io.h libsvn_subr/stream.c svnadmin/svnadmin.c svnfsfs/load-index-cmd.c tests/libsvn_subr/stream-test.c

2015-09-11 Thread Evgeny Kotkov
Stefan Fuhrmann writes: > So, you are absolutely right about APR buffering being enough for our > non-interactive "read stdin as a single large file" usage in load & friends. > > My last concern is as follows. We need an alternative implementation > for

Re: svn commit: r1700799 - in /subversion/trunk/subversion: include/svn_io.h libsvn_subr/stream.c svnadmin/svnadmin.c svnfsfs/load-index-cmd.c tests/libsvn_subr/stream-test.c

2015-09-11 Thread Evgeny Kotkov
Julian Foad writes: > Also for: >svnrdump load >svnmucc put - > > ? (These just came to mind when I thought about it.) I think, yes. Thank you for pointing this out, Julian. Regards, Evgeny Kotkov

Re: svn commit: r1700799 - in /subversion/trunk/subversion: include/svn_io.h libsvn_subr/stream.c svnadmin/svnadmin.c svnfsfs/load-index-cmd.c tests/libsvn_subr/stream-test.c

2015-09-09 Thread Evgeny Kotkov
Stefan Fuhrmann writes: >> The original commit begins using svn_stream_wrap_buffered_read() during >> svnadmin load-revprops and svnfsfs load-index. This patch, however, does >> something entirely different and adds buffering to *every* stdin. > > Sorry for the

Re: svn commit: r1700799 - in /subversion/trunk/subversion: include/svn_io.h libsvn_subr/stream.c svnadmin/svnadmin.c svnfsfs/load-index-cmd.c tests/libsvn_subr/stream-test.c

2015-09-08 Thread Stefan Fuhrmann
On Mon, Sep 7, 2015 at 6:38 PM, Evgeny Kotkov wrote: > Stefan Fuhrmann writes: > > > That is indeed a good idea, so I tried it in various ways: > > > > Index: subversion/libsvn_subr/stream.c > >

Re: svn commit: r1700799 - in /subversion/trunk/subversion: include/svn_io.h libsvn_subr/stream.c svnadmin/svnadmin.c svnfsfs/load-index-cmd.c tests/libsvn_subr/stream-test.c

2015-09-07 Thread Evgeny Kotkov
Stefan Fuhrmann writes: > That is indeed a good idea, so I tried it in various ways: > > Index: subversion/libsvn_subr/stream.c > === > --- subversion/libsvn_subr/stream.c(revision 1701330) > +++

Re: svn commit: r1700799 - in /subversion/trunk/subversion: include/svn_io.h libsvn_subr/stream.c svnadmin/svnadmin.c svnfsfs/load-index-cmd.c tests/libsvn_subr/stream-test.c

2015-09-04 Thread Stefan Fuhrmann
On Wed, Sep 2, 2015 at 3:33 PM, Evgeny Kotkov wrote: > Stefan Fuhrmann writes: > > > Author: stefan2 > > Date: Wed Sep 2 13:04:51 2015 > > New Revision: 1700799 > > > > URL: http://svn.apache.org/r1700799 > > Log: > > [Combines r1698359 and

Re: svn commit: r1700799 - in /subversion/trunk/subversion: include/svn_io.h libsvn_subr/stream.c svnadmin/svnadmin.c svnfsfs/load-index-cmd.c tests/libsvn_subr/stream-test.c

2015-09-02 Thread Evgeny Kotkov
Stefan Fuhrmann writes: > Author: stefan2 > Date: Wed Sep 2 13:04:51 2015 > New Revision: 1700799 > > URL: http://svn.apache.org/r1700799 > Log: > [Combines r1698359 and r170078 into a single commit for better review.] > Introduce a stream wrapper object that adds buffering