Skip to content

Conversation

@cofin
Copy link
Member

@cofin cofin commented Nov 11, 2025

Introduce the StatementStack and StackOperation for managing SQL execution sequences, along with a StackExecutionError for error handling. Update configuration validation to reject class instances. Enhance driver integration and add comprehensive unit tests across various adapters, including edge cases and performance metrics. Document the new Query Stack features and provide an example for usage.

… SQL execution sequences

feat(exceptions): add StackExecutionError for handling stack operation failures
feat(driver): integrate stack execution in async and sync drivers
test: add unit tests for StatementStack and StackResult functionalities
…adapters

- Implement tests for StatementStack sequential execution and continue-on-error behavior in BigQuery, DuckDB, Oracle, Psqlpy, Psycopg, and SQLite adapters.
- Introduce edge case tests for StatementStack execution in SQLite, ensuring proper handling of empty stacks and mixed operations.
- Enhance OracleAsyncDriver with pipeline support tests, verifying native pipeline execution and error handling.
- Add metrics tracking for stack execution to monitor performance and error rates.
- Create a utility script to run pre-commit hooks without requiring PTY support.
…ate related references

- Changed all instances of StackResult.raw_result to StackResult.result across the codebase.
- Updated the StackResult class to reflect the new naming convention, including adjustments to its constructor and methods.
- Modified adapter implementations and tests to accommodate the new result handling.
- Ensured backward compatibility in tests by verifying the correct behavior of result handling in various database adapters.
@cofin cofin merged commit 2cb5dfe into main Nov 14, 2025
22 of 24 checks passed
@cofin cofin deleted the feat/sqlstack branch November 14, 2025 22:35
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