generated from JetBrains/intellij-platform-plugin-template
-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
command /database for CRUD and SQL #171
Comments
phodal
added a commit
that referenced
this issue
Dec 30, 2024
Added `executeSqlQuery` method in `DatabaseSchemaAssistant` to execute SQL queries. Introduced `Execute` enum value in `DatabaseFunctionProvider` to handle SQL query requests. This extends the database tool's functionality to directly execute SQL statements.
phodal
added a commit
that referenced
this issue
Dec 30, 2024
…ndling #171 - Replace verbose script model creation with a concise null-coalescing operator. - Add logic to handle null console cases by creating and running query runners.
phodal
added a commit
that referenced
this issue
Dec 30, 2024
Improved console retrieval logic by adding fallback to create a new console if none is active. Also streamlined query execution flow by ensuring console is properly initialized before use.
phodal
added a commit
that referenced
this issue
Dec 30, 2024
Replace direct query execution with message bus-based processing for better extensibility. Added support for handling grid data requests and consumer callbacks. Removed unused file editor retrieval code.
phodal
added a commit
to shire-lang/shire-demo
that referenced
this issue
Dec 30, 2024
phodal
added a commit
that referenced
this issue
Dec 30, 2024
Move `CompletableFuture` creation outside the thread execution block and adjust result completion logic to check row size instead of waiting for the last row. Added a TODO for further improvement.
phodal
added a commit
that referenced
this issue
Dec 30, 2024
- Split single import statement for `com.intellij.database.datagrid.*` into individual imports. - Remove commented-out code related to `CompletableFuture` and `ApplicationManager`. - Change `DGDepartment.DATA_IMPORT` to `DGDepartment.DATA_EXPORT` for clarity.
phodal
added a commit
that referenced
this issue
Dec 30, 2024
- Simplify `getDataSources` method to use single-expression syntax. - Rename `getAllRawDatasource` to `allRawDatasource` for consistency. - Remove unused `Disposer` import. - Fix minor formatting issues
phodal
added a commit
that referenced
this issue
Dec 31, 2024
Refactored the database runner execution logic to use `invokeAndWait` and `runSingleRunner` for better error handling and code clarity. Removed redundant code and improved the flow of database session management.
phodal
added a commit
that referenced
this issue
Dec 31, 2024
Extracted SQL execution logic into separate functions (`executeSqlInConsole` and `executeSql`) for better readability and maintainability. Removed unused imports and redundant code. Simplified the flow of SQL execution and error handling.
phodal
added a commit
that referenced
this issue
Dec 31, 2024
Include DATABASE command in the list of commands that trigger code block checks. Additionally, wrap database runner retrieval in a read action to ensure
phodal
added a commit
that referenced
this issue
Dec 31, 2024
Ensure thread safety by using `runReadAction` for PSI file retrieval. Added error handling for cases where the PSI file is not found, returning a descriptive error message instead of throwing an exception.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the solution you'd like
For example
The text was updated successfully, but these errors were encountered: