Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Commit 80b0344

Browse files
authored
Merge pull request #64 from datafuselabs/fix/additional_headers
fix: additional_headers
2 parents e368aba + 6eb198b commit 80b0344

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

databend_py/connection.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,11 @@ def query(self, statement):
164164
try:
165165
resp_dict = self.do_query(url, query_sql)
166166
self.client_session = resp_dict.get("session", self.default_session())
167-
self.additional_headers = {XDatabendQueryIDHeader: resp_dict.get(QueryID)}
167+
if self.additional_headers:
168+
self.additional_headers.update(
169+
{XDatabendQueryIDHeader: resp_dict.get(QueryID)})
170+
else:
171+
self.additional_headers = {XDatabendQueryIDHeader: resp_dict.get(QueryID)}
168172
return self.wait_until_has_schema(resp_dict)
169173
except Exception as err:
170174
log.logger.error(

tests/test_client.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -199,9 +199,11 @@ def test_null_to_none(self):
199199
self.assertIsNone(data[0][0])
200200

201201
def test_set_query_id_header(self):
202+
os.environ["ADDITIONAL_HEADERS"] = "X-DATABENDCLOUD-TENANT=TENANT,X-DATABENDCLOUD-WAREHOUSE=WAREHOUSE"
202203
client = Client.from_url(self.databend_url)
204+
self.assertEqual("X-DATABENDCLOUD-TENANT" in client.connection.additional_headers, True)
205+
self.assertEqual(client.connection.additional_headers["X-DATABENDCLOUD-TENANT"], "TENANT")
203206
client.execute("select 1")
204-
print(client.connection.additional_headers)
205207
execute_query_id1 = client.connection.additional_headers["X-Databend-Query-Id"]
206208
self.assertEqual("X-Databend-Query-Id" in client.connection.additional_headers, True)
207209
client.execute("select 2")

0 commit comments

Comments
 (0)