OctopusDeploy / Halibut

| Public | A secure communication stack for .NET using JSON-RPC over SSL.

Other

12 stars 44 forks source link

Remove the Message property from the LogEvent to reduce the size of the LogEvent object #483

Closed nathanwoctopusdeploy closed 1 year ago

nathanwoctopusdeploy commented 1 year ago

Background

Currently Server retains up to 100 LogEvents per ExecutionTarget which quickly grows to 100's MB when scaling Server.

The LogMesage object contains a redundant field Message that is not referenced by Server. Remove this redundant field to reduce some Memory usage.

Example Memory Usage

Example Server Usage of LogEvent

Related to https://github.com/OctopusDeploy/Issues/issues/8266

How to review this PR

Quality :heavy_check_mark:

Pre-requisites

[ ] I have read How we use GitHub Issues for help deciding when and where it's appropriate to make an issue.
[ ] I have considered informing or consulting the right people, according to the ownership map.
[ ] I have considered appropriate testing for my change.