turicas/rows

Optimize if output format is SQLite

Offen

#217 geöffnet am 20.02.2017

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (136 Forks)github user discovery
clienhancementhelp wanted

Repository-Metriken

Stars
 (886 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

If we run the following command:

rows query 'SELECT * FROM table1 WHERE field1="value1"' filename.sqlite --output=out.sqlite

the CLI will:

  • Connect to a SQLite database named filename.sqlite since there's only one source and it's format is SQLite
  • Import the SQLite result into a rows.Table running the query
  • Export the result rows.Table into a new SQLite database named out.sqlite

We can optimize this use case replacing the steps with the following:

  • Connect to a SQLite database named filename.sqlite since there's only one source and its format is SQLite
  • Attach the output database with SQLite's ATTACH SQL command
  • Change the query to have INSERT INTO (to add result to the new database)
  • Run the query

More details on ATTACH in StackOverflow this thread.

Contributor Guide

Research-Richtung
Untersuche die Verwendung von SQLite ATTACH DATABASE, um den Export und erneuten Import der Tabelle zu vermeiden; ändere die Abfrage, um INSERT INTO mit der angehängten Datenbank zu verwenden.
Tech Stack
python
Domain
backend
Issue Type
Performance
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
PythonSQLSQLite
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60