Ticket #9106 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

RemoveLogs should be able to keep some logs

Reported by: Andrei Savici Owned by: Andrei Savici
Priority: minor Milestone: Release 3.2
Component: Framework Keywords:
Cc: Blocked By:
Blocking: #9105 Tester: Peter Peterson

Description

When creating MD workspaces from multiple files, the size of the logs becomes too big. However, RemoveLogs will delete all logs, even those that might be useful later. We should introduce a whitelist, such as the logs on that list are not deleted.

Change History

comment:1 Changed 7 years ago by Nick Draper

  • Status changed from new to assigned

together with a booleam property on the algorithm "ExcludeWhitelist"

comment:2 Changed 7 years ago by Andrei Savici

  • Status changed from assigned to inprogress

Add option to keep some logs in RemoveLogs. Refs #9106

Changeset: c9fe52a6a66b49e3b05f771223bc3a5984ccda5a

comment:3 Changed 7 years ago by Andrei Savici

  • Status changed from inprogress to verify
  • Resolution set to fixed

Branch feature/9106_RemoveLogsWhiteList To test: load any file. Run RemoveLogs with certain logs that you want to keep. The unit test uses CNCS7860. I choose Speed3 (a time series property with several values) and gd_prtn_chrg (a number property). Check if you have the logs that you wanted, and only those

comment:4 Changed 7 years ago by Russell Taylor

  • Status changed from verify to reopened
  • Resolution fixed deleted

Loading files in unit tests is not allowed. It is easy to add some logs manually to a workspace - look at SumEventsByLogValueTest for an example.

comment:5 Changed 7 years ago by Andrei Savici

  • Status changed from reopened to inprogress

Removed LoadNexusLog dependency. Refs #9106

Changeset: 35f2bcb5777f8353cc56322156a0be5a97c6d5d5

comment:6 Changed 7 years ago by Andrei Savici

  • Status changed from inprogress to verify
  • Resolution set to fixed

comment:7 Changed 7 years ago by Peter Peterson

  • Status changed from verify to verifying
  • Tester set to Peter Peterson

comment:8 Changed 7 years ago by Peter Peterson

  • Status changed from verifying to closed

Merge remote branch 'origin/feature/9106_RemoveLogsWhiteList'

Full changeset: 7d5da4eaf148aadf2afea6c33f28ae777361bc27

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 9949

Note: See TracTickets for help on using tickets.