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

Add support for syncing more complex paths #35

Merged
merged 1 commit into from
Oct 14, 2024

Conversation

gcampbell512
Copy link
Contributor

It is currently difficult to sync on a small portion of a model if that model includes a list element. For example /node1/node2//node3/node4. The current method would be to select node1/node2/ then exclude the unwanted fields. This however can cause substantial processing.

This change allows paths of type /node1/node2/*/node3/node4 to be entered. The new code uses apteryx_query and apteryx_watch_tree to retrieve the database information to synchronize.

Additionally exclude paths are now only applied to the model they were added with.

It is currently difficult to sync on a small portion of a model if that
model includes a list element. For example /node1/node2/*/node3/node4.
The current method would be to select node1/node2/* then exclude the
unwanted fields. This however can cause substantial processing.

This change allows paths of type /node1/node2/*/node3/node4 to be
entered. The new code uses apteryx_query and apteryx_watch_tree
to retrieve the database information to synchronize.

Additionally exclude paths are now only applied to the model they were
added with.
@sparlane sparlane requested a review from carlgsmith October 13, 2024 22:13
@carlgsmith carlgsmith requested a review from sparlane October 13, 2024 22:13
@sparlane sparlane merged commit d630574 into alliedtelesis:master Oct 14, 2024
1 check passed
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

Successfully merging this pull request may close these issues.

2 participants