theseus-rs / rsql

Command line interface for CockroachDB, DuckDB, LibSQL, MariaDB, MySQL, PostgreSQL, Redshift, Snowflake, SQLite3 and SQL Server
https://theseus-rs.github.io/rsql/rsql_cli/
Apache License 2.0
129 stars 5 forks source link

refactor: re-use database metadata between drivers for the same database #123

Closed brianheineman closed 7 months ago

github-actions[bot] commented 7 months ago

🐰Bencher

ReportSun, April 21, 2024 at 00:23:10 UTC
Projecttheseus-rs/rsql
Branchrefactor-metadata
Testbedlocalhost
BenchmarkLatencyLatency Results
nanoseconds (ns)
duckdb➖ (view plot)4004100.000
postgres-embedded➖ (view plot)818350000.000
postgresql-embedded➖ (view plot)822930000.000
rusqlite➖ (view plot)263830.000
sqlite➖ (view plot)501850.000

Bencher - Continuous Benchmarking
View Public Perf Page
Docs | Repo | Chat | Help
codecov[bot] commented 7 months ago

Codecov Report

Attention: Patch coverage is 93.75000% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 95.83%. Comparing base (2eb8c87) to head (0926837).

Files Patch % Lines
rsql_drivers/src/metadata.rs 97.72% 1 Missing :warning:
rsql_drivers/src/postgres/driver.rs 0.00% 1 Missing :warning:
rsql_drivers/src/rusqlite/driver.rs 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #123 +/- ## ========================================== - Coverage 95.94% 95.83% -0.11% ========================================== Files 87 85 -2 Lines 9618 9445 -173 ========================================== - Hits 9228 9052 -176 - Misses 390 393 +3 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.