Skip to content

Add Databricks SQL Warehouse and MongoDB connection support#15

Open
jiatolentino wants to merge 2 commits into
datamasque:mainfrom
jiatolentino:databricks-mongodb-connections
Open

Add Databricks SQL Warehouse and MongoDB connection support#15
jiatolentino wants to merge 2 commits into
datamasque:mainfrom
jiatolentino:databricks-mongodb-connections

Conversation

@jiatolentino

Copy link
Copy Markdown
Collaborator

Summary

The connections command hand-maintained its own ConnectionType enum and class map, which had fallen behind datamasque-python. Databricks SQL Warehouse and MongoDB both ship dedicated config classes in the library (DatabricksConnectionConfig, MongoConnectionConfig), and list/get already deserialized them, but neither could be created or imported via the CLI.

This wires both types into the CLI so they can be created from a JSON file (--file), consistent with how snowflake and dynamodb are handled (too many fields for flags).

Changes

  • Add databricks and mongodb to the ConnectionType enum and _CONNECTION_CLASSES map.
  • Import DatabricksConnectionConfig and MongoConnectionConfig from datamasque-python.
  • Update the --type help text and add a Databricks --file example to the create docstring.
  • Add from-file creation tests for both types.
  • Bump version to 1.4.0 and update CHANGELOG.md.

Testing

  • make check passes (ruff lint + format, mypy, 218 tests).
  • Verified end-to-end against a live DataMasque instance: create, list, get, and delete round-trip for both Databricks and MongoDB connections, with secrets correctly redacted on read.

@jiatolentino jiatolentino self-assigned this Jun 14, 2026
@jiatolentino jiatolentino requested a review from carlosfunk June 14, 2026 21:46
@carlosfunk

Copy link
Copy Markdown
Member

I see @kw-datamasque has an open PR (#14) to change the release process so you might want to wait for that first and remove your changelog entry...

@kw-datamasque

Copy link
Copy Markdown
Collaborator

I see @kw-datamasque has an open PR (#14) to change the release process so you might want to wait for that first and remove your changelog entry...

This PR is still in draft and being tested so may be a short while until its merge in so free to edit the changelog here and I'll tidy it all once it's merged in 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants