Closed alexkh-db closed 3 months ago
Attention: Patch coverage is 48.71795%
with 20 lines
in your changes are missing coverage. Please review.
Project coverage is 57.66%. Comparing base (
a714146
) to head (b95d3c4
).
Files | Patch % | Lines |
---|---|---|
databricks/sdk/config.py | 48.71% | 20 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Changes
We are currently adding serverless support to Databricks Connect client. Serverless can be enabled in the client using
DatabricksSession.builder.serverless()
. We also want to add support for serverless in the databricks config.compute
attribute in the config that will be new unifying field that can contain values"serverless"
or"cluster:<cluster_id>"
.cluster_id
will be deprecated but still supported after this change. We could add support forwarehouse_id
in thecompute
field as well if necessary.compute
andcluster_id
should be mutually exclusive, if both are specified as kwargs, env variables or fields in.databrickscfg
, an error will be thrown.get_compute
method returns aCompute
object that parses.compute
and.cluster_id
fields and provides convinientis_empty()
,is_serverless()
, andget_cluster_id()
methods.Tests
make test
run locallymake fmt
applied