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.

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

[FEATURE]: Add support for migrate-tables-ctas in cli cmd databricks labs ucx migrate-tables #1659

Closed
1 task done
HariGS-DB opened this issue May 7, 2024 · 0 comments · Fixed by #1660
Closed
1 task done
Labels
enhancement New feature or request needs-triage

Comments

@HariGS-DB
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

Problem statement

The current cli cmd for migrate-tables handles migrate-tables and migrate-external-hiveserde-tables-in-place-experimental workflow but doesn't cover migrate-external-tables-ctas

Proposed Solution

Add logic to check the requirement to run migrate-external-tables-ctas workflow and prompt user to run it

Additional Context

No response

@HariGS-DB HariGS-DB added enhancement New feature or request needs-triage labels May 7, 2024
@HariGS-DB HariGS-DB added this to UCX May 7, 2024
@github-project-automation github-project-automation bot moved this to Triage in UCX May 7, 2024
@nfx nfx closed this as completed in #1660 May 7, 2024
nfx pushed a commit that referenced this issue May 7, 2024
…abs ucx migrate-tables` (#1660)

## Changes
This change adds support for migrate-tables-ctas into the existing cli
cmd for migrate-tables.
Checks for the presence of an external table which cannot be synced and
prompts the user to run the additional workflow
Also updated relevant readme doc



Resolves #1659 

### Functionality 

- [X] added relevant user documentation
- [ ] added new CLI command
- [X] modified existing command: `databricks labs ucx ...`
- [ ] added a new workflow
- [ ] modified existing workflow: `...`
- [ ] added a new table
- [ ] modified existing table: `...`

### Tests
<!-- How is this tested? Please see the checklist below and also
describe any other relevant tests -->

- [ ] manually tested
- [X] added unit tests
- [ ] added integration tests
- [ ] verified on staging environment (screenshot attached)
@github-project-automation github-project-automation bot moved this from Triage to Archive in UCX May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs-triage
Projects
Archived in project
1 participant