cedar-policy / cedar

Implementation of the Cedar Policy Language
https://www.cedarpolicy.com
Apache License 2.0
889 stars 80 forks source link

Generate actions HashMap at ValidatorSchema instantiation #1290

Closed tpaulus closed 3 weeks ago

tpaulus commented 4 weeks ago

Description of changes

This change fixes part of the issues identified in #1285 by pulling up the expensive action inversion from CoreSchema construction to ValidatorSchema construction, which is only done once per Schema.

Checklist for requesting a review

The change in this PR is (choose one, and delete the other options):

I confirm that this PR (choose one, and delete the other options):

I confirm that cedar-spec (choose one, and delete the other options):

I confirm that docs.cedarpolicy.com (choose one, and delete the other options):