Infer values from child notebook loaded via dbutils.notebook.run() call

databrickslabs / ucx

Your best companion for upgrading to Unity Catalog. UCX will guide you, the Databricks customer, through the process of upgrading your account, groups, workspaces, jobs etc. to Unity Catalog.

Other

196 stars 70 forks source link

Infer values from child notebook loaded via dbutils.notebook.run() call #2083

Closed ericvergnaud closed 6 days ago

ericvergnaud commented 1 week ago

Changes

When linting notebooks, use AST from child notebooks loaded via dbutils.notebook.run() to infer values

Linked issues

Progresses #1205 Progresses #1901

Functionality

None

Tests

[x] added unit tests

marginally improves not computed (-9/744) when running make solacc

github-actions[bot] commented 1 week ago

✅ 160/160 passed, 1 flaky, 6 skipped, 3h39m15s total

Flaky tests:

🤪 test_experimental_permissions_migration_for_group_with_same_name (2m15.958s)

_{Running from acceptance #4548}

ericvergnaud commented 6 days ago

@nfx is something still blocking this PR?

ericvergnaud commented 6 days ago

closing in favor of #2091