Skip to content

Commit fac2560

Browse files
docs: clarify async kernel rowcount requirement
Signed-off-by: Aakash Saravanan <aakash.saravanan@databricks.com>
1 parent de947bf commit fac2560

2 files changed

Lines changed: 3 additions & 0 deletions

File tree

‎CHANGELOG.md‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
# Unreleased
44
- Fix inline parameters (`use_inline_params=True`) so a sequence directly after `IN` renders as a list again: `x IN %(ids)s` renders as `x IN (1,2,3)` instead of `x IN ARRAY(1,2,3)`, which Spark rejects. Inside the list, tuples render as rows for a multi-column `IN`, and a `set` or `frozenset` is accepted again. Sequences anywhere else still render as `ARRAY(...)` ([#629](https://github.com/databricks/databricks-sql-python/issues/629))
5+
- Report `cursor.rowcount` after asynchronous DML completes. With `use_kernel=True`, this requires a kernel release containing async affected-row count support; kernel 1.4.0 continues to report `-1`.
56

67
# 4.9.0 (2026-11-04)
78
- Upgrade Databricks SQL Kernel to 1.4.0, which keeps Lakehouse Real-Time session affinity and flushes shared telemetry only after the final session closes.

‎src/databricks/sql/backend/kernel/client.py‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -967,6 +967,8 @@ def get_execution_result(
967967
# result-set construction fails after a successful await.
968968
try:
969969
result_set = self._make_result_set(stream, cursor, command_id)
970+
# Older kernel wheels do not expose this value. Keep them compatible
971+
# while allowing a newer binding to populate async rowcount.
970972
result_set.num_modified_rows = getattr(handle, "num_modified_rows", None)
971973
return result_set
972974
except Exception as exc:

0 commit comments

Comments
 (0)