The GitHub Actions job "Commit Check" on jackrabbit-oak.git/issue/OAK-12348 has 
failed.
Run started by GitHub user bhabegger (triggered by bhabegger).

Head commit for run:
2013864ad685f8afc1f4c16744c34c0071371110 / Benjamin Habegger 
<[email protected]>
OAK-12348: remove PropertyIndexPlan fields only used in one place

matchesAllTypes, matchesNodeTypes, properties, pathFilter, unique, and
bestCount were all read only inside the constructor or inside a helper
method called only from the constructor -- none needed to survive as
instance state. useLegacy is now consumed once, in the constructor, to
compute cost, so it no longer needs to be a field either.

getCostLegacy/getCostConfigurable/getStrategies now take bestCount/unique
as parameters instead of reading fields, since they're never called
outside the constructor.

Report URL: https://github.com/apache/jackrabbit-oak/actions/runs/33598834425

With regards,
GitHub Actions via GitBox

Reply via email to