Skip to content

Commit b80cb71

Browse files
committed
Close cursors before closing connection
Signed-off-by: Jesse Whitehouse <[email protected]>
1 parent 5ae9486 commit b80cb71

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

src/databricks/sql/client.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -181,12 +181,11 @@ def close(self) -> None:
181181
self._close()
182182

183183
def _close(self, close_cursors=True) -> None:
184-
self.thrift_backend.close_session(self._session_handle)
185-
self.open = False
186-
187184
if close_cursors:
188185
for cursor in self._cursors:
189186
cursor.close()
187+
self.thrift_backend.close_session(self._session_handle)
188+
self.open = False
190189

191190
def commit(self):
192191
"""No-op because Databricks does not support transactions"""

0 commit comments

Comments
 (0)