Commit 85cc283
committed
fix: move stdlib json import to module level in hot-path functions
Hoists `import json` from inside hot-path function bodies to module
top in two services, matching the pattern from #1216 (which fixed
#1209 for `copy` and `datetime` in app/nodes/).
- app/services/tracer_client/tracer_integrations.py
- get_all_integrations: import was inside a `for` loop iterating
integration records, re-running the import statement once per
iteration
- get_grafana_credentials: import was per-call during integration
resolution
- app/services/lambda_client.py
- invoke_function: import was per Lambda invocation
Fixes #12421 parent ae53af1 commit 85cc283
2 files changed
Lines changed: 2 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
| 4 | + | |
4 | 5 | | |
5 | 6 | | |
6 | 7 | | |
| |||
325 | 326 | | |
326 | 327 | | |
327 | 328 | | |
328 | | - | |
329 | | - | |
330 | 329 | | |
331 | 330 | | |
332 | 331 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
| 5 | + | |
5 | 6 | | |
6 | 7 | | |
7 | 8 | | |
| |||
65 | 66 | | |
66 | 67 | | |
67 | 68 | | |
68 | | - | |
69 | | - | |
70 | 69 | | |
71 | 70 | | |
72 | 71 | | |
| |||
98 | 97 | | |
99 | 98 | | |
100 | 99 | | |
101 | | - | |
102 | | - | |
103 | 100 | | |
104 | 101 | | |
105 | 102 | | |
| |||
0 commit comments