databricks / cli

Databricks CLI
Other
148 stars 56 forks source link

Fix panic in `apply_presets.go` #1796

Closed lennartkats-db closed 1 month ago

lennartkats-db commented 1 month ago

Changes

This fixes the user-reported panic in apply_presets.go. I'm still unsure how to reproduce this, since the CLI just reports ob broken_job is not defined when I try to use bundle deploy with an empty job. That said — we may as well be defensive here and I see we have lots of checks for empty job/cluster/etc. settings scattered throughout our code base so at least we're somewhat consistent.