tree 3706b16dce56f39c5179c63ac74d499fd899e16d
parent 655e4ab18a6736c4f902a5a73b6d3bd61daebef0
author Patrick Hiesel <hiesel@google.com> 1676379150 +0100
committer Patrick Hiesel <hiesel@google.com> 1676379915 +0000

Only validate non-empty queries that do not match all changes

Empty and * queries will just make the reviewer filter apply
in later processing. So they are always valid.

This change is a latency improvement since trying to construct a
query with `*` on servers that have account visibility checking will
cause Gerrit to lookup many accounts (through ChangeQueryBuilder ->
default -> parseAccount -> AccountResolver -> check).

Google-Bug: b/267202984
Change-Id: If9f5e879b05044e23b527d65838da6900a378956