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

Re: serf property caching

From: Hyrum K. Wright <hyrum_wright_at_mail.utexas.edu>
Date: 2007-08-10 23:27:07 CEST

Justin Erenkrantz wrote:
> On 8/8/07, Hyrum K. Wright <hyrum_wright@mail.utexas.edu> wrote:
...
>> explicitly asking for it, but rather the set of checked_in_props. (This
>> set does *not* contain DAV:href, and I don't know why the initial call
>> to retrieve_props() succeeds.) When I use all_props instead of
>> checked_in_props, the DAV:href gets pulled from the cache correctly, and
>> we don't hit the abort().
>
> Instead of using all_props, I'd just add a new structure to ra_serf.h:
>
> static const svn_ra_serf__dav_props_t href_props[] =
> {
> { "DAV:", "href" },
> { NULL }
> };
>
> and use href_props rather than all_props.

I made this change, and it works. However, other calls to
svn_ra_serf__retrieve_props() are failing with a different message:
subversion/libsvn_ra_serf/util.c:329: (apr_err=20014)
svn: Error running context: Internal error

This looks like some kind of error internal to serf itself. In doing
more digging, it looks to be coming from context.c, around line 870.
Given that information, I think that this is a bug in serf, not in our
usage of it. My serf-fu isn't that great, though, so I'm not sure. I'm
going to commit my patch with Justin's recommended change above, but it
will generate quite a number of new failures. Anybody with a little bit
more knowledge care to debug this new problem?

(FWIW, log_tests 3 is a simple test which easily generates this error.)

-Hyrum

Received on Fri Aug 10 23:25:58 2007

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