ucx icon indicating copy to clipboard operation
ucx copied to clipboard

Fix `create-catalogs-schemas` to allow more than 1 level nesting more than the external location

Open mohanab-db opened this issue 1 year ago • 1 comments

Changes

Fix create-catalogs-schemas to allow more than 1 level nesting more than the external location.

Linked issues

Resolves #1700

Functionality

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

Tests

  • [ ] manually tested
  • [ ] added unit tests
  • [ ] added integration tests
  • [ ] verified on staging environment (screenshot attached)

mohanab-db avatar May 15 '24 21:05 mohanab-db

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 89.41%. Comparing base (ab03d5c) to head (1e2d18b). Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1701   +/-   ##
=======================================
  Coverage   89.41%   89.41%           
=======================================
  Files          93       93           
  Lines       11383    11384    +1     
  Branches     2002     2002           
=======================================
+ Hits        10178    10179    +1     
  Misses        822      822           
  Partials      383      383           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar May 16 '24 09:05 codecov[bot]

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

:white_check_mark: mohanab-db
:x: JCZuurmond
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar May 21 '24 06:05 CLAassistant