Skip to content

Snowflake Connection Optimizations#652

Open
sagarswamirao wants to merge 5 commits intomainfrom
sagark/snowflake-debug
Open

Snowflake Connection Optimizations#652
sagarswamirao wants to merge 5 commits intomainfrom
sagark/snowflake-debug

Conversation

@sagarswamirao
Copy link
Copy Markdown
Collaborator

No description provided.

…ectionExplorer component

- Added `fetchTableSchema` query parameter to the API, allowing users to control whether per-table column metadata is loaded.
- Updated the ConnectionExplorer component to use a new state management approach for selected tables, improving clarity and functionality.
- Refactored the TableSchemaViewer and introduced a new SelectedTableDetailPanel for better separation of concerns and enhanced loading state handling.
- Adjusted API documentation to reflect the new query parameter and its behavior.

These changes improve the user experience by providing more control over data fetching and enhancing the component structure for maintainability.

Signed-off-by: Sagar Swami Rao Kulkarni <sagarswamirao@gmail.com>
- Updated the version of `@malloydata/malloy-interfaces` in package.json and bun.lock to "^0.0.368".
- Increased the versions of various `@malloydata` database packages in bun.lock and server package.json to "^0.0.368".

These changes ensure compatibility with the latest updates and improvements in the Malloy data packages.

Signed-off-by: Sagar Swami Rao Kulkarni <sagarswamirao@gmail.com>
- Updated the SQL query in the listTablesForSchema function to use `SHOW TABLES IN SCHEMA ${schemaName}` for better compatibility with Snowflake.
- Enhanced logging to include debug information about listed tables.
- Improved the mapping of result rows to ensure proper handling of table names, accommodating variations in casing.

These changes enhance the functionality and maintainability of the database utility functions.

Signed-off-by: Sagar Swami Rao Kulkarni <sagarswamirao@gmail.com>
@sagarswamirao sagarswamirao requested a review from kylenesbit April 2, 2026 04:53
- Changed the default value of a boolean parameter from `false` to `true` in the API documentation for improved clarity on expected behavior.

This update ensures that users have accurate information regarding the default settings when interacting with the API.

Signed-off-by: Sagar Swami Rao Kulkarni <sagarswamirao@gmail.com>
…nsistency

- Removed an unnecessary boolean parameter from the SelectedTableDetailPanel function.
- Adjusted the formatting of conditional checks in the TableSchemaViewer and TablesInSchema functions for better clarity.
- Enhanced the overall structure of the code to maintain consistency in style and improve maintainability.

These changes contribute to a cleaner codebase and enhance the readability of the ConnectionExplorer component.

Signed-off-by: Sagar Swami Rao Kulkarni <sagarswamirao@gmail.com>
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.

1 participant