Skip to content

Optimize service release logger memory use - #73

Merged
awarson2233 merged 4 commits into
mainfrom
worktree-agent-abea5c5acd2ec548f
Jun 10, 2026
Merged

awarson2233 merged 4 commits into
mainfrom
worktree-agent-abea5c5acd2ec548f

Conversation

@awarson2233

Copy link
Copy Markdown
Owner

What changed

  • Switch Release logging to a synchronous file-only logger path guarded by .
  • Avoid initializing the spdlog async thread pool / async logger for the Service Release logger, removing the async queue allocation.
  • Make project / respect , with Service Release set to info-level裁剪 and Debug preserving trace logging.
  • Keep Debug logging behavior as async + console + file + optional , and avoid forcing App/Tools Release loggers into Service file-only mode.

Why

Service Release does not use App/DVR/IPC debug logging paths, so the async logger queue is unnecessary runtime memory overhead for the production service. This reduces baseline private memory while preserving Debug diagnostics.

Verification

  • ninja: no work to do.
  • -- Checking/Generating .clangd config for clangd language server...
    -- Auto-generated .clangd config successfully at: D:/source/repos/EGoTouchRev-rebuild/.clangd
    -- Link-Time Optimization (LTO/IPO) is supported. Enabling for Release configuration...
    -- Build spdlog: 1.17.0
    -- Build type: Debug
    -- Configuring done (5.9s)
    -- Generating done (0.5s)
    -- Build files have been written to: D:/source/repos/EGoTouchRev-rebuild/build/arm64-Debug
  • [1/8] Linking CXX static library spdlog_build\spdlogd.lib
    [2/8] Linking CXX static library Common.lib
    [3/8] Linking CXX static library IPCCore.lib
    [4/8] Linking CXX static library Solvers.lib
    [5/8] Linking CXX static library Host.lib
    [6/8] Linking CXX static library EGoTouchServiceCore.lib
    [7/8] Linking CXX static library Device.lib
    [8/8] Linking CXX executable EGoTouchService.exe

🤖 Generated with Claude Code

awarson2233 and others added 4 commits June 11, 2026 00:30
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@awarson2233
awarson2233 merged commit 2e5fbc5 into main Jun 10, 2026
1 check passed
@awarson2233
awarson2233 deleted the worktree-agent-abea5c5acd2ec548f branch June 10, 2026 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant