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

Stop spewing UUIDs #807

Closed
wants to merge 9 commits into from
Closed

Stop spewing UUIDs #807

wants to merge 9 commits into from

Conversation

yinlinchen
Copy link
Contributor

@ajs6f
Copy link
Contributor

ajs6f commented Jun 2, 2015

It's clear from this PR that we will no longer be testing to prove that we do spew UUIDs, but will we no longer actually be spewing them? I see no changes to the actual RDF generation machinery here...

@yinlinchen
Copy link
Contributor Author

What file are you refer to? I thought remove JCR_UUID in
fcrepo-kernel/src/main/java/org/fcrepo/kernel/FedoraJcrTypes.java is the
change need to be done for this ticket...

On Tue, Jun 2, 2015 at 9:21 AM, A. Soroka notifications@github.com wrote:

It's clear from this PR that we will no longer be testing to prove that we
do spew UUIDs, but will we no longer actually be spewing them? I see no
changes to the actual RDF generation machinery here...


Reply to this email directly or view it on GitHub
#807 (comment).

@ajs6f
Copy link
Contributor

ajs6f commented Jun 2, 2015

No, you need to make sure that the triple in question actually stops being published. I don't know where it is created, but it's probably somewhere near PropertiesRdfContext.

@awoods
Copy link

awoods commented Jul 2, 2015

Superseded by: #827

@awoods awoods closed this Jul 2, 2015
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

4 participants