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

[FEATURE]: Rewrite usages of DBFS Mount paths in Python code, including /dbfs/mnt local fs, dbfs:/, and /mnt #1109

Open
1 task done
Tracked by #1204
nfx opened this issue Mar 25, 2024 · 2 comments · May be fixed by #3476
Open
1 task done
Tracked by #1204

[FEATURE]: Rewrite usages of DBFS Mount paths in Python code, including /dbfs/mnt local fs, dbfs:/, and /mnt #1109

nfx opened this issue Mar 25, 2024 · 2 comments · May be fixed by #3476
Assignees
@pritishpai
Labels
migrate/code Abstract Syntax Trees and other dark magic migrate/volumes migrate from raw DBFS mounts to UC Volumes

Comments

@nfx
Copy link
Collaborator

nfx commented Mar 25, 2024 •
edited
Loading

Is there an existing issue for this?

  • I have searched the existing issues

Problem statement

Do best effort AST/CST rewrite for UC-migrated locations

Proposed Solution

Build on top of #1103

Additional Context

Related
@nfx nfx added the migrate/code Abstract Syntax Trees and other dark magic label Mar 25, 2024
@nfx nfx added this to UCX Mar 25, 2024
@github-project-automation github-project-automation bot moved this to Triage in UCX Mar 25, 2024
@ericvergnaud ericvergnaud mentioned this issue Apr 1, 2024
Merged
11 tasks
This was referenced Apr 1, 2024
Open
Closed
@nfx nfx moved this from Triage to Month Backlog in UCX Apr 22, 2024
@nfx nfx added the migrate/volumes migrate from raw DBFS mounts to UC Volumes label Apr 24, 2024
@nfx nfx moved this from Month Backlog to Quarter Backlog in UCX Apr 24, 2024
@nfx nfx moved this from Quarter Backlog to Active Backlog in UCX Jul 4, 2024
@JCZuurmond
Copy link
Member

@nfx : Why on top of #1103? My suggestion is to separate the streaming out as long as it is not requested by customers. We do need to detect the mount paths before rewriting them, maybe a generic approach is possible that covers streaming as well

@JCZuurmond
Copy link
Member

Looks like a duplicate of #392, could someone confirm and close #392 if that is the case?

@nfx nfx moved this from Active Backlog to Design in UCX Jul 17, 2024
@nfx nfx mentioned this issue Apr 22, 2024
Open
@gueniai gueniai moved this to Todo in UCX Dec 6, 2024
@pritishpai pritishpai moved this from Todo to In Progress in UCX Dec 20, 2024
@pritishpai pritishpai linked a pull request Jan 6, 2025 that will close this issue
Draft
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pritishpai pritishpai

Labels
migrate/code Abstract Syntax Trees and other dark magic migrate/volumes migrate from raw DBFS mounts to UC Volumes
Projects
UCX
Status: In Progress
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Rewrite dbfs mount path usages to Unity Catalog tables databrickslabs/ucx
3 participants
@nfx @JCZuurmond @pritishpai