Skip to content

Commit e1c7091

Browse files
simplify logging, comments
Signed-off-by: varun-edachali-dbx <[email protected]>
1 parent 9f11c9d commit e1c7091

File tree

1 file changed

+4
-7
lines changed

1 file changed

+4
-7
lines changed

src/databricks/sql/backend/utils/http_client.py

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,6 @@ def _make_request(
114114
elif method.upper() == "POST":
115115
response = self.session.post(url, headers=headers, json=data)
116116
elif method.upper() == "DELETE":
117-
# For DELETE requests, use params for data (query parameters)
118117
response = self.session.delete(url, headers=headers, params=data)
119118
else:
120119
raise ValueError(f"Unsupported HTTP method: {method}")
@@ -128,14 +127,12 @@ def _make_request(
128127
# Parse JSON response
129128
if response.content:
130129
result = response.json()
130+
131131
# Log response content (but limit it for large responses)
132132
content_str = json.dumps(result, indent=4, sort_keys=True)
133-
if len(content_str) > 1000:
134-
logger.debug(
135-
f"Response content (truncated): {content_str[:1000]}..."
136-
)
137-
else:
138-
logger.debug(f"Response content: {content_str}")
133+
content_str = content_str[:1000] + "..." if len(content_str) > 1000 else content_str
134+
logger.debug(f"Response content: {content_str}")
135+
139136
return result
140137
return {}
141138

0 commit comments

Comments
 (0)