-
Notifications
You must be signed in to change notification settings - Fork 4
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
FCR API must define PUT-to-create-objects #11
Comments
@cbeer Can you say a little more about this? |
LDP says:
I think our API ought to require support for PUT. |
I think I caught @barmintor advocating remaining silent, last week. My feeling is if Fedora requires PUT creation, it should specify clear behavior for handling containments for resources created in this way. But I think I'm fine with remaining silent, too. The current draft says: "LDP-RS SHOULD support PUT to update user data". So at this stage, we aren't even floating requiring PUT at all. |
@ajs6f Well, for one, how else are we going to create containers in the first place? Or, do we require the root of the repository is itself a container? |
@cbeer Currently we do make that happen in the ref impl, and I have no problem with writing that into the spec. On the other hand, one of the issues that @no-reply and I and others discussed around this was in what way the repository does and does not have to be connected via LDP containment. We came to the rough conclusion in an IRC chat that the multitenancy use case suggests that the repository does not have to be entirely connected, which would suggest that PUT be available in order to create new roots/tenants. But that discussion didn't include you, so I want to understand your thinking on this. |
Discussion at LDCX was generally against PUT for create, and against repositories without contiguous containment. The main reason given for this was that node discovery is basically impossible in this case. For my part, this seems like an orthogonal issue. REST generally doesn't define discovery, and I'm not sure an LDP server in general, or Fedora in particular, needs to either. |
Not totally clear what the phrase "node discovery" means in this context-- are we talking about how to find out where a repository exists? |
@ajs6f "node" here meaning "LDP Resource". Sorry for lack of clarity. Given a Fedora instance, resources connected to the repository base will be navigable via containment. Any free floating resources would not. |
In that case I agree with you, @no-reply. This reminds me of my (weak cheese) "Google crawler" use case. IIRC, in that conversation you and I came to the conclusion that an instance of Fedora may usefully include several "regions"/tenants/workspaces/staging areas/whatever-you-want-to-call-them, each one being a containment-connected subgraph, without the full repository being containment-connected. I still think that's true. I'd like to hear more about the use cases for node discovery. Are we still talking about crawlers? |
As we bring the initial phase of spec writing to a close, it would be helpful if y'all (especially @cbeer and @no-reply) could opine at the current place for discussion. I myself am very inclined to the idea of create-on-PUT, but it's not clear to me that we must require it in the spec. Maybe you can help? |
I think if we require it than we must also specify the containment and On Thu, Nov 3, 2016 at 10:35 AM, A. Soroka [email protected] wrote:
|
My sense of those issues is straightforward-- no containment required (meaning an impl MUST NOT do auto-containment, and multiple, uncontainment-connected regions within a server are fine) and no intervening node-creation required (impls MAY produce intervening nodes). Is that problematic? |
Obviously, FCR API needs a stronger requirement.
The text was updated successfully, but these errors were encountered: