Skip to content

Add chaos testing scripts and verification scenarios for SQLite and SQL Server#82

Closed
isdaniel wants to merge 1 commit into
mainfrom
chore/add-sqlserver-sqlite-integration-test
Closed

Add chaos testing scripts and verification scenarios for SQLite and SQL Server#82
isdaniel wants to merge 1 commit into
mainfrom
chore/add-sqlserver-sqlite-integration-test

Conversation

@isdaniel
Copy link
Copy Markdown
Owner

  • Introduced scenario5_verify.sql to verify high-volume data replication (3M rows) in SQL Server.
  • Created run_chaos_tests_sqlite.sh for executing chaos tests with SQLite as the destination.
  • Developed run_chaos_tests_sqlserver.sh for executing chaos tests with SQL Server as the destination.
  • Implemented run_pgbench_chaos_test_sqlite.sh for performance testing with pgbench on SQLite.
  • Implemented run_pgbench_chaos_test_sqlserver.sh for performance testing with pgbench on SQL Server.
  • Added logging and cleanup functionalities to all scripts for better traceability and resource management.

…QL Server

- Introduced `scenario5_verify.sql` to verify high-volume data replication (3M rows) in SQL Server.
- Created `run_chaos_tests_sqlite.sh` for executing chaos tests with SQLite as the destination.
- Developed `run_chaos_tests_sqlserver.sh` for executing chaos tests with SQL Server as the destination.
- Implemented `run_pgbench_chaos_test_sqlite.sh` for performance testing with pgbench on SQLite.
- Implemented `run_pgbench_chaos_test_sqlserver.sh` for performance testing with pgbench on SQL Server.
- Added logging and cleanup functionalities to all scripts for better traceability and resource management.
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request significantly expands the project's testing infrastructure by introducing comprehensive chaos and performance (pgbench) testing suites for SQL Server and SQLite destinations. Key changes include updating the Dockerfile to support feature-based builds, extending the Makefile with specialized test targets, and adding various docker-compose configurations and shell scripts to orchestrate the testing environments. The review feedback highlights a critical mismatch in the container shutdown timeout that could lead to data loss, identifies security vulnerabilities regarding cleartext passwords in connection strings and command-line arguments, and suggests performance optimizations for SQL Server schema definitions and shell scripting best practices.

Comment thread tests/chaos/scripts/chaos_script.sh
Comment thread tests/chaos/scripts/run_chaos_tests_sqlserver.sh
Comment thread tests/chaos/scripts/run_pgbench_chaos_test_sqlserver.sh
Comment thread tests/chaos/scripts/run_pgbench_chaos_test_sqlserver.sh
Comment thread examples/scripts/init_sqlserver.sql
@isdaniel isdaniel closed this Apr 27, 2026
@isdaniel isdaniel deleted the chore/add-sqlserver-sqlite-integration-test branch April 27, 2026 14:34
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