Skip to content

[DATABRICKS-1] PLU-600: add databricks config, env vars, and package dependency#1526

Open
pregnantboy wants to merge 1 commit intotrunk/databricksfrom
databricks/1-config
Open

[DATABRICKS-1] PLU-600: add databricks config, env vars, and package dependency#1526
pregnantboy wants to merge 1 commit intotrunk/databricksfrom
databricks/1-config

Conversation

@pregnantboy
Copy link
Copy Markdown
Contributor

@pregnantboy pregnantboy commented Apr 7, 2026

Add Databricks integration support

Add Databricks environment variables to ECS config and .env-example, install @databricks/sql package, and create config module with zod validation for Databricks credentials.

Changes

  • Added five new Databricks environment variables to ECS configuration:

    • DATABRICKS_SERVER_HOSTNAME
    • DATABRICKS_HTTP_PATH
    • DATABRICKS_CATALOG
    • DATABRICKS_CLIENT_ID
    • DATABRICKS_CLIENT_SECRET
  • Installed @databricks/sql package version 1.12.0 for Databricks connectivity

  • Created new Databricks configuration module with zod schema validation to ensure all required credentials are present

  • Updated backend .env-example with Databricks environment variable placeholders

  • Added Databricks config import to main app environment variables index

@linear
Copy link
Copy Markdown

linear bot commented Apr 7, 2026

@pregnantboy pregnantboy changed the title PLU-600 : add databricks config, env vars, and package dependency [DATABRICKS-1] PLU-600: add databricks config, env vars, and package dependency Apr 7, 2026
@pregnantboy pregnantboy marked this pull request as ready for review April 7, 2026 07:01
@pregnantboy pregnantboy requested a review from a team as a code owner April 7, 2026 07:01
Comment thread packages/backend/.env-example
@pregnantboy pregnantboy force-pushed the databricks/1-config branch 2 times, most recently from 0da7c5f to d33c9d8 Compare April 13, 2026 03:08
Copy link
Copy Markdown
Contributor

@kevinkim-ogp kevinkim-ogp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Add Databricks environment variables to ECS config and .env-example,
install @databricks/sql package, and create config module with zod
validation for Databricks credentials.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@pregnantboy pregnantboy force-pushed the databricks/1-config branch from d33c9d8 to 8629e4e Compare April 16, 2026 08:44
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.

2 participants