tree 7e7ea30cf2fb4fb6b6804b9934e963cee2102e7a
parent 80d90858ad98c4e274439a6fe628a4260a28cfdf
author Luca Milanesio <luca.milanesio@gmail.com> 1646153525 +0000
committer Luca Milanesio <luca.milanesio@gmail.com> 1646993786 +0000

Change visibility: prefer test instead of check

Do not rely to throwing AuthExceptions for checking
change visibility but ForChange.test instead, which
returns a boolean.

Relying to exceptions to be thrown and catching them
for a simple change filtering would cause the JVM to
trigger multiple context switching and slowing down
the overall ACL evaluation.

A slow ACL evaluation for changes filtering may
result in the increase of threads utilisation
while the change query is executed.

Release-Notes: skip
Change-Id: Ie488a41b28b5c19710a11c2840793f04b1d3861b