Skip to content

Commit ddc61a0

Browse files
correct some merge artifacts
Signed-off-by: varun-edachali-dbx <[email protected]>
1 parent f4f27e3 commit ddc61a0

File tree

2 files changed

+2
-12
lines changed

2 files changed

+2
-12
lines changed

src/databricks/sql/client.py

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -782,14 +782,6 @@ def execute(
782782
async_op=False,
783783
enforce_embedded_schema_correctness=enforce_embedded_schema_correctness,
784784
)
785-
self.active_result_set = ResultSet(
786-
self.connection,
787-
execute_response,
788-
self.backend,
789-
self.buffer_size_bytes,
790-
self.arraysize,
791-
self.connection.use_cloud_fetch,
792-
)
793785

794786
if self.active_result_set.is_staging_operation:
795787
self._handle_staging_operation(

tests/unit/test_client.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -102,10 +102,8 @@ def test_closing_connection_closes_commands(self, mock_result_set_class):
102102
# Close the connection
103103
connection.close()
104104

105-
self.assertTrue(
106-
mock_result_set_class.return_value.has_been_closed_server_side
107-
)
108-
mock_result_set_class.return_value.close.assert_called_once_with()
105+
# Check that the manually created mock result set's close method was called
106+
mock_result_set.close.assert_called_once_with()
109107

110108
@patch("%s.session.ThriftDatabricksClient" % PACKAGE_NAME)
111109
def test_cant_open_cursor_on_closed_connection(self, mock_client_class):

0 commit comments

Comments
 (0)