Re: Authz perf regression 1.9 -> 1.10

2019-09-12 Thread Sam Toliman
Many thanks for feedback! It was rather a quick example to confirm the possibility to optimize this function in my case.

Re: Authz perf regression 1.9 -> 1.10

2019-09-12 Thread Branko Čibej
On 12.09.2019 13:30, Sam Toliman wrote: > >> (~3500k in my case) > > You have more than 3 million groups? > Misprinted. There are ~3500 groups in my case. > > > The best thing you can do is prepare a patch, with tests, that > > implements your suggestions. Considering of course that correctness is

Re: Authz perf regression 1.9 -> 1.10

2019-09-12 Thread Sam Toliman
>> (~3500k in my case) > You have more than 3 million groups? Misprinted. There are ~3500 groups in my case. > The best thing you can do is prepare a patch, with tests, that > implements your suggestions. Considering of course that correctness is > more important than performance. :) I will try to

Re: Authz perf regression 1.9 -> 1.10

2019-09-12 Thread Branko Čibej
On 12.09.2019 10:44, Sam Toliman wrote: > > How do you propose that the parser should know, in advance, which parts > > of the authz and groups files it can discard unless it parses all of > them? > > By this point >

Re: Authz perf regression 1.9 -> 1.10

2019-09-12 Thread Sam Toliman
> How do you propose that the parser should know, in advance, which parts > of the authz and groups files it can discard unless it parses all of them? By this point b->client_info

Re: Authz perf regression 1.9 -> 1.10

2019-09-11 Thread Branko Čibej
On 11.09.2019 18:34, Sam Toliman wrote: > > Greetings, > > TLDR since 1.10 subversion loading acl file ~15x times slower, which > leads in performance regression in tunnel mode used with svn+ssh scheme. > > https://subversion.apache.org/docs/release-notes/1.10#authzperf says > that performance was