Skip to content

[DATABRICKS-3] PLU-600: add dynamic data queries, dynamic actions, and GraphQL mutation#1528

Open
pregnantboy wants to merge 1 commit intodatabricks/2-app-skeletonfrom
databricks/3-dynamic-data
Open

[DATABRICKS-3] PLU-600: add dynamic data queries, dynamic actions, and GraphQL mutation#1528
pregnantboy wants to merge 1 commit intodatabricks/2-app-skeletonfrom
databricks/3-dynamic-data

Conversation

@pregnantboy
Copy link
Copy Markdown
Contributor

@pregnantboy pregnantboy commented Apr 7, 2026

Add Databricks table and column management functionality

Add list-table-names, list-table-columns, create-table, and create-column dynamic data handlers for Databricks integration. Add IDynamicAction type to support action-based dynamic operations, DynamicActionInput GraphQL schema for mutation parameters, and dynamicAction mutation resolver to execute dynamic actions with proper validation and error handling.

Copy link
Copy Markdown
Contributor Author

pregnantboy commented Apr 7, 2026

@pregnantboy pregnantboy changed the title feat: add dynamic data queries, dynamic actions, and GraphQL mutation [DATABRICKS-3] PLU-600: add dynamic data queries, dynamic actions, and GraphQL mutation Apr 7, 2026
@linear
Copy link
Copy Markdown

linear bot commented 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
@datadog-opengovsg
Copy link
Copy Markdown

datadog-opengovsg bot commented Apr 7, 2026

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: ea3ac20 | Docs | Was this helpful? Give us feedback!

@pregnantboy pregnantboy force-pushed the databricks/2-app-skeleton branch from 7651d1b to ed3204a Compare April 7, 2026 09:43
@pregnantboy pregnantboy force-pushed the databricks/3-dynamic-data branch from 9df0e2b to 23f55f5 Compare April 7, 2026 09:43
@pregnantboy pregnantboy force-pushed the databricks/2-app-skeleton branch from ed3204a to c7de60f Compare April 13, 2026 03:08
@pregnantboy pregnantboy force-pushed the databricks/3-dynamic-data branch from 23f55f5 to 6e6fe2e Compare April 13, 2026 03:08
Comment thread packages/backend/src/apps/databricks/dynamic-data/create-column.ts Outdated
Comment thread packages/backend/src/graphql/mutations/dynamic-action.ts
@pregnantboy pregnantboy force-pushed the databricks/3-dynamic-data branch from 6e6fe2e to 62c5d53 Compare April 13, 2026 03:30
Comment thread packages/backend/src/apps/databricks/dynamic-data/create-column.ts
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, tested that dynamic data and dynamic actions work!

only thing is to address the graphite comment in create-column.ts so that the BadUserInputError is thrown as intended

@pregnantboy pregnantboy force-pushed the databricks/2-app-skeleton branch from c7de60f to f03d4f1 Compare April 16, 2026 08:44
@pregnantboy pregnantboy force-pushed the databricks/3-dynamic-data branch from 62c5d53 to e8e971b Compare April 16, 2026 08:44
Add list-table-names, list-table-columns, create-table, and
create-column dynamic data handlers. Add IDynamicAction type,
DynamicActionInput GraphQL schema, and dynamicAction mutation resolver.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@pregnantboy pregnantboy force-pushed the databricks/3-dynamic-data branch from e8e971b to ea3ac20 Compare April 17, 2026 06:32
@pregnantboy pregnantboy force-pushed the databricks/2-app-skeleton branch from f03d4f1 to 6741ce2 Compare April 17, 2026 06:32
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