Thread (5 messages) 5 messages, 4 authors, 2025-07-01

Re: [PATCH 2/2] bloom: enable multiple pathspec bloom keys

From: SZEDER Gábor <hidden>
Date: 2025-07-01 08:50:58

On Fri, Jun 27, 2025 at 06:50:22AM -0700, Junio C Hamano wrote:
Totally outside the topic, but I wonder if we can further optimize
by adding an early rejection using .nowildcard_len?  Instead of
allowing a wildcarded "dir/*" pathspec element from disabling the
Bloom filter altogether, we could say "dir/ is not possibly altered,
so there may be dir/A, dir/B, etc., in the directory, nothing that
would match dir/* wildcard would have been modified", couldn't we?
Indeed, that's what I demonstrated back in:

  https://public-inbox.org/git/20200529085038.26008-35-szeder.dev@gmail.com/
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help