Skip to content

feat: add centralized execution time logging#124

Open
vkryzh wants to merge 1 commit intomasterfrom
feat/execution-time-logging
Open

feat: add centralized execution time logging#124
vkryzh wants to merge 1 commit intomasterfrom
feat/execution-time-logging

Conversation

@vkryzh
Copy link

@vkryzh vkryzh commented Mar 3, 2026

Summary

  • Add centralized execution time logging to the SDK handler wrapper
  • Logs total data processing time in milliseconds after every handler invocation
  • Covers all app types (stream, scheduled, task, partial_rerun_merge) automatically
  • No changes needed in individual apps

Changes

  • Import time module
  • Capture time.time() at handler entry in base_handler()
  • Log elapsed time via CORVA_LOGGER.info() in a finally block

Test plan

  • Verify all existing tests pass
  • Deploy to a test environment and confirm "Total data processing time: X ms" appears in logs
  • Verify timing is logged even when handler raises an exception

@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

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