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

Fix for FCREPO-1678 #881

Merged
merged 1 commit into from Aug 11, 2015
Merged

Fix for FCREPO-1678 #881

merged 1 commit into from Aug 11, 2015

Conversation

ajs6f
Copy link
Contributor

@ajs6f ajs6f commented Aug 11, 2015

No description provided.

@acoburn
Copy link
Contributor

acoburn commented Aug 11, 2015

I am noticing that a request for a NonRdfSourceDescription returns the same ETag value as the NonRdfSource. This isn't an issue for caching proxies, as the resources correspond to different URLs. I also notice, however, that when the NonRdfSourceDescription changes (e.g. via sparql-update), the ETag header for the NonRdfSource also changes. Is that the intended behavior? The bits on the NonRdfSource surely have not changed, so why the new ETag value? This is certainly better than the current behavior -- where a sparql-update on the description changes the ETag on the Binary, but not the description.

@ajs6f
Copy link
Contributor Author

ajs6f commented Aug 11, 2015

That is the correct behavior. The description and the bitstream have an independent existence in HTTP-land, although we had rather they did not. (You will recall that after much thought and argument, it wasn't clear how to merge them.) But they do not have an independent existence as resources in the repository. They each live exactly as long as the other. The properties in question are properties of the resource persisted as the bitstream, not some nebulous and ill-defined "metadata node".

acoburn added a commit that referenced this pull request Aug 11, 2015
@acoburn acoburn merged commit 54cc3a1 into master Aug 11, 2015
@acoburn acoburn deleted the FCREPO-1678 branch August 11, 2015 20:42
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.

None yet

2 participants