search

temporalio / sdk-python

Temporal Python SDK
MIT License
457 stars 68 forks source link

Updated lazy loaded logger_details #593

Closed TheCodeWrangler closed 2 months ago

TheCodeWrangler commented 2 months ago

What was changed

This PR updates the lazy loading behavior for activity _logger_details so that activity details will be included in logging outputs even if not preloaded by adding to message.

Solution identified/proposed by @cretz in this Issue: https://github.com/temporalio/sdk-python/issues/592

Why?

Desire to allow activity logs to include extra information without adding them to the message string.

Checklist

  1. Closes 592

  2. How was this tested: Manually running logs

  3. Any docs updates needed? No

CLAassistant commented 2 months ago

CLA assistant check
All committers have signed the CLA.