-
Notifications
You must be signed in to change notification settings - Fork 9
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
GetSystemMetadata method doesn't get replica information #140
Comments
there is no field for |
Yeah, I omitted that replica field because, at the time we were discussing removing it from sysmeta altogether, and it's not client editable so I skipped it. Is it needed for editing, or just for completeness? |
For completeness. I think it should show the replica information.
Otherwise, it may confuse users in some circumstances.
Also Metacat should enforce that clients can't edit the replica parts when
they call MN.updateSystemMetadata:
NCEAS/metacat#1867
…On Thu, Apr 4, 2024 at 3:47 PM Matt Jones ***@***.***> wrote:
Yeah, I omitted that replica field because, at the time we were discussing
removing it from sysmeta altogether, and it's not client editable so I
skipped it. Is it needed for editing, or just for completeness?
—
Reply to this email directly, view it on GitHub
<#140 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB5QQDATAD5QRS3P74BSOTDY3XJ6ZAVCNFSM6AAAAABFXX476WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMZYGQYDAMZWGQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I used a piece of R code to get system metadata of an object which has three replicas.
The system metadata can be see from:
https://mn-sandbox-ucsb-1.test.dataone.org/knb/d1/mn/v2/meta/testReplicateJsonLD.1711738610440
However, the replica section is missing from R even though it got the replication policy part:
The text was updated successfully, but these errors were encountered: