mirror of https://github.com/databricks/cli.git
e4cd782852
## Changes - Remove bundle.Parallel & bundle.ReadOnlyBundle. - Add bundle.ApplyParallel, as a helper to migrate from bundle.Parallel. - Keep ReadOnlyMutator as a separate type but it's now a subtype of Mutator so it works on regular *Bundle. Having it as a separate type prevents non-readonly mutators being passed to ApplyParallel - validate.Validate becomes a function (was Mutator). ## Why This a follow up to #2390 where we removed most of the tools to construct chains of mutators. Same motivation applies here. When it comes to read-only bundles, it's a leaky abstraction -- since it's a shallow copy, it does not actually guarantee or enforce readonly access to bundle. A better approach would be to run parallel operations on independent narrowly-focused deep-copied structs, with just enough information to carry out the task (this is not implemented here, but the eventual goal). Now that we can just write regular code in phases and not limited to mutator interface, we can switch to that approach. ## Tests Existing tests. --------- Co-authored-by: shreyas-goenka <88374338+shreyas-goenka@users.noreply.github.com> |
||
|---|---|---|
| .. | ||
| generate | ||
| loader | ||
| mutator | ||
| resources | ||
| validate | ||
| variable | ||
| artifact.go | ||
| artifacts_test.go | ||
| bundle.go | ||
| deployment.go | ||
| experimental.go | ||
| filename.go | ||
| filename_test.go | ||
| git.go | ||
| lock.go | ||
| lock_test.go | ||
| presets.go | ||
| resources.go | ||
| resources_test.go | ||
| root.go | ||
| root_test.go | ||
| sync.go | ||
| target.go | ||
| workspace.go | ||
| workspace_test.go | ||