SNOW-870802: make log_telemetry_imported_packages threadsafe (#1665)

sfc-gh-aalam · web-flow · commit 7fa66ba913c6 · 2023-08-24T16:41:21.000-04:00
diff --git a/src/snowflake/connector/connection.py b/src/snowflake/connector/connection.py
@@ -1725,7 +1725,7 @@ def _log_telemetry_imported_packages(self) -> None:
             # and internal modules with names starting with an underscore
             imported_modules = {
                 k.split(".", maxsplit=1)[0]
-                for k in sys.modules.keys()
+                for k in list(sys.modules)
                 if not k.startswith("_")
             }
             ts = get_time_millis()

Original file line number	Diff line number	Diff line change
`@@ -1725,7 +1725,7 @@ def _log_telemetry_imported_packages(self) -> None:`
`1725`	`1725`	`# and internal modules with names starting with an underscore`
`1726`	`1726`	`imported_modules = {`
`1727`	`1727`	`k.split(".", maxsplit=1)[0]`
`1728`		`- for k in sys.modules.keys()`
	`1728`	`+ for k in list(sys.modules)`
`1729`	`1729`	`if not k.startswith("_")`
`1730`	`1730`	`}`
`1731`	`1731`	`ts = get_time_millis()`