Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update historical data processing logs #232

Merged
merged 1 commit into from
Jul 2, 2024

Conversation

adev4a
Copy link
Collaborator

@adev4a adev4a commented Jul 2, 2024

PR Details

Update logs for historical data processing service

Description

Adds additional logs to the historical data processing service to log records after the influx write command is called on the record.
The information from these logs is intended to be used for data analysis to calculate the processing time for the rosbags.

Related GitHub Issue

Related Jira Key

WFD-577

Motivation and Context

How Has This Been Tested?

Types of changes

  • Defect fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that cause existing functionality to change)

Checklist:

  • I have added any new packages to the sonar-scanner.properties file
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@adev4a adev4a self-assigned this Jul 2, 2024
@adev4a adev4a requested a review from dan-du-car July 2, 2024 17:21
@adev4a adev4a changed the title update logs update historical data processing logs Jul 2, 2024
@dan-du-car dan-du-car merged commit 939beb1 into develop Jul 2, 2024
16 checks passed
@dan-du-car dan-du-car deleted the wfd-577-update-hdp-for-performance-metrics branch July 2, 2024 19:10
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.

2 participants