Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[TODO] bring back self.tables_migrator.index() #3153

Open
Tracked by #2074
nfx opened this issue Nov 4, 2024 · 0 comments · May be fixed by #3744
Open
Tracked by #2074

[TODO] bring back self.tables_migrator.index() #3153

nfx opened this issue Nov 4, 2024 · 0 comments · May be fixed by #3744
Assignees
@pritishpai
Labels
tech debt chores and design flaws

Comments

@nfx
Copy link
Collaborator

nfx commented Nov 4, 2024

See:

ucx/src/databricks/labs/ucx/contexts/application.py

Lines 503 to 508 in b97a8d5

self.path_lookup,
TableMigrationIndex([]), # TODO: bring back self.tables_migrator.index()
self.directfs_access_crawler_for_paths,
self.used_tables_crawler_for_paths,
self.config.include_job_ids,
)

@nfx nfx added the tech debt chores and design flaws label Nov 4, 2024
@github-project-automation github-project-automation bot moved this to Triage in UCX Nov 4, 2024
@JCZuurmond JCZuurmond marked this as a duplicate of #3154 Feb 25, 2025
@JCZuurmond JCZuurmond mentioned this issue Feb 25, 2025
Closed
@JCZuurmond JCZuurmond moved this to Ready for Review in UCX Feb 25, 2025
@JCZuurmond JCZuurmond self-assigned this Feb 25, 2025
This was referenced Feb 25, 2025
Draft
Open
@JCZuurmond JCZuurmond assigned pritishpai and unassigned JCZuurmond Mar 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pritishpai pritishpai

Labels
tech debt chores and design flaws
Projects
UCX
Status: Ready for Review
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Pass table migration index to WorkflowLinter databrickslabs/ucx
3 participants
@nfx @JCZuurmond @pritishpai