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

Allow Partial Parentage #101

Merged
merged 2 commits into from
Oct 23, 2023
Merged

Conversation

d-ylee
Copy link
Contributor

@d-ylee d-ylee commented Sep 27, 2023

For Issue #94:
These changes allow Partial Parentage to be provided. In order to provide partial parentage, the client must provide a missing_files field in the JSON Body. missing_files is used to validate that the number of child-parent file id pairs that do not have parentage are purposefully provided by each client. If a client provides an incorrect number of missing_files, then the request will fail. missing_files will default to 0, preserving the previous behavior.

…d issue with parentage check

Kept validation check
Partial Parentage is indicated by providing -1 to ChildParentIDList
Only insert valid parentage
@d-ylee d-ylee changed the title Issue #94: Allow Partial Parentage Allow Partial Parentage Sep 27, 2023
@amaltaro
Copy link

@d-ylee Hi Dennis, I am working on the relevant changes on the WMAgent side, basically:

  • providing -1 as parent file id instead of None/null
  • counting the number of files that do not have ANY parent file and provide it with the new argument missing_files

I should have those ready today, such that we can proceed with some basic tests.

@amaltaro
Copy link

@d-ylee Dennis, from my side we can go ahead and merge this pull request and make it available in testbed for further testing. Thank you for preparing it.

@d-ylee d-ylee merged commit 4d184ca into dmwm:master Oct 23, 2023
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