Optimization: Reuse existing Data Models in `AnalyzerReport`

[lakshita10341]

Problem

Currently, the AnalyzerReport.create_data_model method creates a new BaseDataModel instance every time it is called, even if an identical data model already exists in the database. This leads to redundant data storage and unnecessary object creation.

There is an existing TODO in api_app/analyzers_manager/models.py highlighting this:

# TODO we don't need to actually crate a new object every time.
#  if the report is the same of the previous one, we can just link it

Proposed Solution

Modify create_data_model to:

1. Construct the data model dictionary as usual.
2. Query the database for an existing data model of the correct class that matches the fields in the dictionary.
3. If a match is found, reuse the existing object.
4. If no match is found, proceed to create a new one.

I would like to work on it. Please let me know if there are any changes needed for this approach.

[lakshita10341]

Hi @mlodic PTAL whenever you get time if these changes are needed or not.

[mlodic]

Query the database for an existing data model of the correct class that matches the fields in the dictionary.

If this is done improperly, it would cause unecessary queries and possible problems. This optimization is kinda risky this can be added only if a good solution is actually found.

You are free to work on it and try a solution but be aware that could not be accepted easily.

[lakshita10341]

Thanks for the feedback!. I'll make sure the query doesn't introduce unnecessary overhead.