Update Analyzer code to catch the table permission issue - Githubissues

airbnb / chronon

Chronon is a data platform for serving for AI/ML applications.

Apache License 2.0

673 stars 36 forks source link

Update Analyzer code to catch the table permission issue #756

Closed yuli-han closed 2 months ago

yuli-han commented 2 months ago

Summary

We have found cases where the analyzer could not catch the table permission issue. This PR it trying to fix the issue.

Why / Goal

Test Plan

[ ] Added Unit Tests
[ ] Covered by existing CI

[x] Integration tested Tested with Airbnb internal trust join

[2024-05-01 00:13:10] {Analyzer:351} - ---- Table permission check completed. Found permission errors in 1 tables ----
[2024-05-01 00:13:10] {Analyzer:352} - trust.no_access_table

Checklist

[ ] Documentation update

Reviewers

@SophieYu41