Skip to content

Commit 45c0f75

Browse files
docs: correct _disable_pandas Kernel support in CONNECTION_PARAMETERS.md
_disable_pandas is a Python-side Arrow->row conversion toggle, not a wire option. KernelResultSet inherits the shared _convert_arrow_table, which reads connection.disable_pandas, so the flag is honored on the kernel path too (the kernel returns Arrow that flows through the same conversion). Mark the row Kernel-supported and drop it from the "ignored on Kernel" summary list. Co-authored-by: Isaac Signed-off-by: eric-wang-1990 <e.wang@databricks.com>
1 parent 4d21062 commit 45c0f75

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

CONNECTION_PARAMETERS.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ to change without notice.
137137
| `use_cloud_fetch` | `bool` ||| `True` | Download large result sets in parallel from cloud storage. The kernel manages result transport internally. |
138138
| `max_download_threads` | `int` ||| `10` | Worker threads for cloud-fetch downloads. Not forwarded to the kernel. |
139139
| `enable_query_result_lz4_compression` | `bool` ||| `True` | LZ4-compress result payloads. Not forwarded; the kernel handles compression internally. |
140-
| `_disable_pandas` | `bool` || | `False` | Skip the pandas-based Arrow deserialization path. Not forwarded to the kernel. |
140+
| `_disable_pandas` | `bool` || | `False` | Skip the pandas-based Arrow→row deserialization and materialize rows directly with PyArrow. This is a **Python-side** result-conversion toggle, not a wire option: the kernel returns results as Arrow (`RecordBatch`es) and the connector runs the *same* `_convert_arrow_table` for both backends, so the flag is honored on the kernel path too. Affects only row fetches (`fetchone`/`fetchmany`/`fetchall`); the `fetch*_arrow` methods return the Arrow table unchanged regardless of this flag. |
141141
| `_use_arrow_native_complex_types` | `bool` ||| `True` | Return `ARRAY`/`MAP`/`STRUCT` as native Arrow types instead of JSON strings. Forwarded to the kernel. |
142142
| `_use_arrow_native_decimals` | `bool` ||| `True` | Return `DECIMAL` as a native Arrow type instead of a string. Thrift-only. |
143143
| `_use_arrow_native_timestamps` | `bool` ||| `True` | Return `TIMESTAMP` as a native Arrow type instead of a string. Thrift-only. |
@@ -186,7 +186,7 @@ regardless of `use_kernel`.
186186
4. TLS-client-cert *authentication* (`_use_cert_as_auth`) — note the TLS
187187
*transport* options (`_tls_*`) themselves **are** honored on both backends.
188188
5. Result-transport tuning: `use_cloud_fetch`, `max_download_threads`,
189-
`enable_query_result_lz4_compression`, `_disable_pandas`.
189+
`enable_query_result_lz4_compression`.
190190
6. Arrow-native rendering for `_use_arrow_native_decimals` /
191191
`_use_arrow_native_timestamps` (complex types **are** forwarded).
192192
7. `staging_allowed_local_path` (Volume `PUT`/`GET`).

0 commit comments

Comments
 (0)