Skip to content

Commit 9afb215

Browse files
author
Jesse
authored
Revert "[PECO-1083] Updated thrift files and added check for protocol version" (#237)
Reverts #229 as it causes all of our e2e tests to fail on some versions of DBR. We'll reimplement the protocol version check in a follow-up. This reverts commit 241e934.
1 parent 1594261 commit 9afb215

File tree

3 files changed

+324
-3964
lines changed

3 files changed

+324
-3964
lines changed

src/databricks/sql/client.py

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
import requests
66
import json
77
import os
8-
from databricks.sql.thrift_api.TCLIService import ttypes
98

109
from databricks.sql import __version__
1110
from databricks.sql import *
@@ -226,11 +225,6 @@ def __del__(self):
226225
def get_session_id(self):
227226
return self.thrift_backend.handle_to_id(self._session_handle)
228227

229-
def get_session_protocol_version(self):
230-
return self.thrift_backend.extract_protocol_version_from_handle(
231-
self._session_handle
232-
)
233-
234228
def get_session_id_hex(self):
235229
return self.thrift_backend.handle_to_hex_id(self._session_handle)
236230

@@ -507,13 +501,6 @@ def execute(
507501
"""
508502
if parameters is None:
509503
parameters = []
510-
elif (
511-
self.connection.get_session_protocol_version()
512-
< ttypes.TProtocolVersion.SPARK_CLI_SERVICE_PROTOCOL_V8
513-
):
514-
raise Error(
515-
"Parameterized operations are not supported by this server. DBR 14.1 is required."
516-
)
517504
else:
518505
parameters = named_parameters_to_tsparkparams(parameters)
519506

0 commit comments

Comments
 (0)