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

[KW-Search] Store direct parent of every node - Update core api #9220

Open
tdraier opened this issue Dec 9, 2024 · 3 comments
Open

[KW-Search] Store direct parent of every node - Update core api #9220

tdraier opened this issue Dec 9, 2024 · 3 comments
Assignees

Comments

@tdraier
Copy link
Contributor

tdraier commented Dec 9, 2024

In order to get the children of one node we need to store the direct parent of each node in data_sources_nodes table. Although the list of parents is stored and supposedly ordered in parents : ([self, parent, .. root]), this is much more clear and stable to rely on dedicated table. We can still rely on this to backfill data.
Parent field will be optional in db and in the api - only connectors which upserts nodes with hierarchy will have to pass a new parent parameter.

@tdraier
Copy link
Contributor Author

tdraier commented Dec 9, 2024

Note : for the backfill, we need to have #9069 solved so that we store in parent the id in the correct format.

@tdraier tdraier self-assigned this Dec 10, 2024
@philipperolet
Copy link
Contributor

@philipperolet
Copy link
Contributor

As discussed, this card is focused on updating core & front API so they get a new parent field ; for the store we will rely on index creation rather

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants