Skip to content

[ty] Use better datetime format for server logs#19083

Merged
dhruvmanila merged 2 commits intomainfrom
dhruv/server-logging
Jul 2, 2025
Merged

[ty] Use better datetime format for server logs#19083
dhruvmanila merged 2 commits intomainfrom
dhruv/server-logging

Conversation

@dhruvmanila
Copy link
Copy Markdown
Member

This PR improves the timer format for ty server logs to be same as Ruff.

Ref: #16389

@dhruvmanila dhruvmanila added internal An internal refactor or improvement server Related to the LSP server ty Multi-file analysis & type inference labels Jul 2, 2025
@dhruvmanila dhruvmanila enabled auto-merge (squash) July 2, 2025 04:25
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 2, 2025

mypy_primer results

Changes were detected when running on open source projects
pegen (https://github.com/we-like-parsers/pegen)
- TOTAL MEMORY USAGE: ~45MB
+ TOTAL MEMORY USAGE: ~41MB

pydantic (https://github.com/pydantic/pydantic)
- TOTAL MEMORY USAGE: ~156MB
+ TOTAL MEMORY USAGE: ~142MB

jinja (https://github.com/pallets/jinja)
- TOTAL MEMORY USAGE: ~106MB
+ TOTAL MEMORY USAGE: ~97MB

paasta (https://github.com/yelp/paasta)
-     memo fields = ~171MB
+     memo fields = ~156MB

isort (https://github.com/pycqa/isort)
- TOTAL MEMORY USAGE: ~80MB
+ TOTAL MEMORY USAGE: ~88MB

tornado (https://github.com/tornadoweb/tornado)
-     memo fields = ~129MB
+     memo fields = ~142MB

static-frame (https://github.com/static-frame/static-frame)
-     memo fields = ~304MB
+     memo fields = ~334MB

openlibrary (https://github.com/internetarchive/openlibrary)
-     memo fields = ~171MB
+     memo fields = ~189MB

rotki (https://github.com/rotki/rotki)
-     memo fields = ~490MB
+     memo fields = ~445MB

@dhruvmanila dhruvmanila merged commit c3d9b21 into main Jul 2, 2025
35 checks passed
@dhruvmanila dhruvmanila deleted the dhruv/server-logging branch July 2, 2025 04:39
dhruvmanila added a commit that referenced this pull request Jul 2, 2025
Follow-up to #19083, also log the
target names like `ty_python_semantic::module_resolver::resolver` in
`2025-07-02 10:12:20.188697000 DEBUG
ty_python_semantic::module_resolver::resolver: Adding first-party search
path '/Users/dhruv/playground/ty_server'` at trace level.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal An internal refactor or improvement server Related to the LSP server ty Multi-file analysis & type inference

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant