Skip to content

Added a "LatestEvents" field to the FetchedInstanceHistory log#1373

Merged
sophiatev merged 4 commits into
mainfrom
stevosyan/add-log-field-to-fetched-instance-history
Jun 23, 2026
Merged

Added a "LatestEvents" field to the FetchedInstanceHistory log#1373
sophiatev merged 4 commits into
mainfrom
stevosyan/add-log-field-to-fetched-instance-history

Conversation

@sophiatev

Copy link
Copy Markdown
Contributor

As per the title, this PR introduces a new log field to log the last 10 events from a fetch instance history call to improve our telemetry.

Copilot AI review requested due to automatic review settings June 23, 2026 19:37

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR extends the Azure Storage tracking telemetry for “FetchedInstanceHistory” by adding a new structured log/ETW field (LatestEvents) that records the last 10 history event types returned by a fetch instance history query.

Changes:

  • Add LatestEvents to the FetchedInstanceHistory structured log event and propagate it through LogHelper and AnalyticsEventSource.
  • Populate LatestEvents from the last 10 event types when reading history entities in AzureTableTrackingStore.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
src/DurableTask.AzureStorage/Tracking/AzureTableTrackingStore.cs Adds computation of the last 10 history event types and passes them into the logger.
src/DurableTask.AzureStorage/Logging/LogHelper.cs Extends FetchedInstanceHistory helper to accept and forward the LatestEvents field.
src/DurableTask.AzureStorage/Logging/LogEvents.cs Adds a new structured field/property (LatestEvents) to the FetchedInstanceHistory log event.
src/DurableTask.AzureStorage/AnalyticsEventSource.cs Extends the ETW FetchedInstanceHistory event payload to include LatestEvents.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/DurableTask.AzureStorage/Tracking/AzureTableTrackingStore.cs
Comment thread src/DurableTask.AzureStorage/AnalyticsEventSource.cs
Comment thread src/DurableTask.AzureStorage/AnalyticsEventSource.cs
Comment thread src/DurableTask.AzureStorage/Logging/LogEvents.cs
Copilot AI review requested due to automatic review settings June 23, 2026 19:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

Comment thread src/DurableTask.AzureStorage/AnalyticsEventSource.cs
@sophiatev sophiatev merged commit 0dad653 into main Jun 23, 2026
47 checks passed
@sophiatev sophiatev deleted the stevosyan/add-log-field-to-fetched-instance-history branch June 23, 2026 21:42
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.

3 participants