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

Re: svn commit: r1617108 - /subversion/branches/authzperf/subversion/libsvn_repos/authz.c

From: Stefan Fuhrmann <stefan.fuhrmann_at_wandisco.com>
Date: Mon, 11 Aug 2014 18:37:47 +0200

On Sun, Aug 10, 2014 at 7:05 PM, Branko Čibej <brane_at_wandisco.com> wrote:

> On 10.08.2014 18:57, Branko Čibej wrote:
>
> On 10.08.2014 16:00, stefan2_at_apache.org wrote:
>
> Author: stefan2
> Date: Sun Aug 10 14:00:08 2014
> New Revision: 1617108
>
> [...]
>
> (has_matching_rule): Update to new data model - just iterate over all
> ACEs and handle generic ACEs as well.
>
>
> Or just use svn_authz__acl_get_access?
>
> When I wrote that code a week ago, this function didn't exist, yet :)

>
> FWIW, you should even have the get_memberships function any more.
>
>
> To clarify: I plan to get rid of svn_authz_tng_t::groups at some point.
> That hash is completely redundant.
>

Careful! This might get us into scalability troubles. But I reserve
judgement until you actually came forth with an implementation.

-- Stefan^2.
Received on 2014-08-11 18:38:15 CEST

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

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