Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix the logic behind matching graph identifiers with the ones coming from the aggregation subsystem #1367

Open
marekhorst opened this issue Jul 12, 2022 · 0 comments

Comments

@marekhorst
Copy link
Member

marekhorst commented Jul 12, 2022

In the #1361 pull request we have replaced the ObjectStore as the source of contents metadata with the hive-based aggregation subsystem.

The identifier matching strategy was simplified, as described in this comment but it turned out we should still involve originalId field when matching the aggregation subsystem entries with the graph results.

This means we should reintroduce the originalId field in the joining operation in order to increase the overlap between the aggregation subsystem payload identifiers and graph result ids.

@marekhorst marekhorst self-assigned this Jul 12, 2022
@marekhorst marekhorst changed the title Fix the logic behing matching graph identifiers with the ones coming from the aggregation subsystem. Fix the logic behind matching graph identifiers with the ones coming from the aggregation subsystem. Jul 12, 2022
@marekhorst marekhorst changed the title Fix the logic behind matching graph identifiers with the ones coming from the aggregation subsystem. Fix the logic behind matching graph identifiers with the ones coming from the aggregation subsystem Jul 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant