fix: only check permission for existing chunks

timescale / doctor

Rule-based recommendations about your timeseries database.

Apache License 2.0

4 stars 3 forks source link

fix: only check permission for existing chunks #30

Closed mkindahl closed 1 year ago

mkindahl commented 1 year ago

If the dropped flag is set for a chunk the real chunk is dropped but still present in the catalog. This leads to errors in the ChunkPermission rule since it tries to check ACL for a non-existing relation.

Fixes #26

mkindahl commented 1 year ago

Fix looks good, it would be nice to have a test for it as well.

Intended to add one after #22 was merged.

mkindahl commented 1 year ago

Building this on #34 so merging now.