etf-validator / governance

ETF Steering Group and the Technical Committee documents
1 stars 2 forks source link

Delete temporary test objects immediately #90

Closed cportele closed 2 years ago

cportele commented 4 years ago

ETF Improvement Proposal (EIP)

Background and Motivation:

Currently test objects and related data are deleted using a queue and resources are deleted with a delay. This behaviour has two drawbacks:

Proposed change

There are two types of test objects:

For each test object three types of data can be distinguished:

For temporary test objects: Delete all source and index data at the end of the test run. The metadata is deleted once the test report is deleted.

For persistent test objects: Delete all source and index data once the persistent test object is deleted. The metadata is deleted, if the persistent test object has been deleted and once the last test report for this test object is deleted.

Alternatives

The current behaviour could have been kept as the default and the new behaviour could have been triggered with an additional flag during the creation of a temporary test object or a test run. However, changing the default behaviour does not have any negative impact on existing users, but avoids a change in the API and keeps the code easier to maintain.

Funding

Yes

Additional information

This could also help with issue #208.

jonherrmann commented 2 years ago

Implemented in Version 2.1.0