Ticket #9106 (closed: fixed)
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: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
together with a booleam property on the algorithm "ExcludeWhitelist"