Skip to content

Refactor Process deduping in the ui converter #713

@yurishkuro

Description

@yurishkuro

The converter currently relies on the caller to ensure that the tags in the Process objects are in the same order, otherwise it won't dedupe them correctly. In many cases the data is passed that violates that assumption (e.g. Cassandra storage creates Process with &model.Process{} instead of model.NewProcess(). It would be better if ui converter did not depend on the behavior of other components and instead would ensure it deals with canonical data.

As part of that change, it's possible that the Process.Hash() function may be no longer needed and should be deleted.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions