Skip to content

Commit c574d39

Browse files
sfc-gh-stakedaankit-bhatnagar167
authored andcommitted
SNOW-78638: Do not send snowflake python connector telemetry during interpreter shutdown
1 parent 5f11bb8 commit c574d39

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

connection.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -499,7 +499,7 @@ def close(self, retry=True):
499499

500500
# close telemetry first, since it needs rest to send remaining data
501501
logger.info('closed')
502-
self._telemetry.close()
502+
self._telemetry.close(send_on_close=retry)
503503
self.rest.delete_session(retry=retry)
504504
self.rest.close()
505505
self._rest = None

telemetry.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -108,10 +108,11 @@ def send_batch(self):
108108
def is_closed(self):
109109
return self._is_closed
110110

111-
def close(self):
111+
def close(self, send_on_close=True):
112112
if not self._is_closed:
113113
logger.debug("Closing telemetry client.")
114-
self.send_batch()
114+
if send_on_close:
115+
self.send_batch()
115116
self._is_closed = True
116117

117118
def disable(self):

0 commit comments

Comments
 (0)