support new Cursor attribute that provides information on completed commands

Offen

#388 erstellt am 25.4.2024 1 Kommentar 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 4/5
Geschätzter Aufwand: 3-5 Tage
Anfängerfreundlichkeit: 35/100

Issue-Typ: Feature
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: python, sql
Bereich: databases

Rechercherichtung

Beginne mit DatabricksConnectionManager.get_response() in dbt-databricks, wo das Issue die hartcodierte "OK"-Nachricht identifiziert, und untersuche den Cursor sowie die Verarbeitung von Befehlergebnissen des Connectors. Vergleiche die gewünschte Ausgabe mit den CommandComplete-Tags von PostgreSQL und den aufgeführten Treiberattributen. Als abgeschlossen gilt die Aufgabe, wenn ausgeführte Befehle ein geeignetes Cursor-Attribut bereitstellen und get_response() die Operation sowie die Anzahl der betroffenen Zeilen meldet.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

enhancement

While a Cursor attribute providing SQL State Code is not officially a part of PEP 249: Python DB API 2.0 spec, it's a common enough convention and would greatly enhance the user experience of dbt-databricks users (👀 @benc-db)

Many database drivers provide this as a Cursor attribute, dbt was able to depend on these drivers to provide it for a ConnectionManager.get_response() method, which reports to users after successful queries the kind of operation performed (SELECT, INSERT, CREATE) and the numbers of rows affected.

However, this is not fully supported today in dbt-databrick, see DatabricksConnectionManager.get_response() where message is hardcoded as "OK" instead of returning more information.

Support for SQL state amongst popular analytics database drivers

Driver	`Cursor` attribute (docs)
psycopg2	`statusmessage`
`snowflake-connector-python`	`sqlstate`

Ideal implementation

Follow Postgres's CommandComplete message

Command	Tag	`rows` indicates the number of rows
`INSERT`	`INSERT 0 rows`	inserted
`DELETE`	`DELETE rows`	deleted
`UPDATE`	`UPDATE rows`	updated
`MERGE`	`MERGE rows`	inserted, updated, or deleted
`SELECT` / `CREATE TABLE AS`	`SELECT rows`	retrieved
`MOVE`	`MOVE rows`	ursor's position has been changed by
`FETCH`	`FETCH rows`	that have been retrieved from the cursor
`COPY`	`COPY rows`	copied, only in `PostgreSQL` 8.2 and later

Vorherrschende Sprache: Python
Sterne: 233
Forks: 152
Ø Merge: 21 Std. 5 Min.
Gemergte PRs (30 T.): 10

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus databricks/databricks-sql-python

`RedactUrlQueryParamsFilter` crashes with `TypeError: NoneType is not iterable` when MLflow's urllib3 filter runs first Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100

databricks/databricks-sql-python#946 · Erstellt am 09.9.2026
Token federation: `_exchange_token` raises KeyError('access_token') on error responses, discarding the real failure reason Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100

databricks/databricks-sql-python#904 · Erstellt am 12.8.2026
[coverage] Conformance findings: PARAMQUERY-022 Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100

databricks/databricks-sql-python#887 · Erstellt am 03.8.2026
[coverage] Conformance findings: METADATA-035 Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100

databricks/databricks-sql-python#886 · Erstellt am 31.7.2026
reviewer-bot: workflow_dispatch reads default-branch code for explore tools (set REVIEW_CONTENT_ROOT) Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100

databricks/databricks-sql-python#865 · Erstellt am 16.7.2026

Alle Issues in databricks/databricks-sql-python