Log of the #duraspace-ff channel on chat.freenode.net

Using timezone: Eastern Standard Time
* jcoyne leaves00:30
* dwilcox joins07:33
* MohamedAR joins08:08
* jmignault joins08:17
* jcoyne joins08:29
* jmignault leaves08:30
<jcoyne>escowles: any ideas what is going on here: https://github.com/fcrepo4/fcrepo4/issues/55108:50
?
<escowles>jcoyne: i don't know why that's happening -- but i've seen the same thing08:51
<pivotal-bot____>Esme Cowles added "Non-RDF resource versioning broken" https://www.pivotaltracker.com/story/show/8090388008:53
<jcoyne>escowles: Good, I'm not crazy.08:59
<escowles>but when i create a new node, fetch the rdf and then PUT it right back, it works for me09:00
so i'm trying to figure out what's going on
* awead joins
<escowles>though if you just want to make the error stop, this commit will do the trick: https://github.com/escowles/fcrepo4/commit/a2b341c36cc9df37d9c6eb0e27ab18c7637779d109:01
jcoyne: the one thing i notice is that the triple that causes the error (primaryType) is missing the datatype09:07
if i download rdf from a node, remove the type, and then post it back, i get that error09:08
so this may just be a typed-literal problem
<jcoyne>escowles: Can you explain in a different way? I'm not catching your meaning.09:10
I don't see any matches for "datatype" in the request/response09:11
<escowles>you're receiving "nt:folder"^^<http://www.w3.org/2001/XMLSchema#string> but posting "nt:folder" for the primaryType
<jcoyne>Got it
escowles++
Aren't those statements equivalent?09:12
<escowles>effectively, but i bet RDF-processing libraries treat them as different because one is "untyped" and the other is typed xsd:string
even though xsd:string is obviously the default type
<jcoyne>I think RDF.rb is discarding the type if it's the default type.09:13
Bah!
<escowles>jcoyne: i think jena does the same thing in java, because i'm seeing the same error (hence that commit above i'm using so i can do ingest testing)09:14
* ajs6f joins09:15
<jcoyne>escowles: It seems like Fedora, as a server, should handle equivalent statements.09:16
<escowles>jcoyne: i agree -- i'm trying to figure out how to do that in jena right now
<ajs6f>jcoyne: I'm not actually sure they are equivalent, or to put it another way, it may depend on what you mean by equivalent (and I'm not trying to be annoying).09:17
jcoyne: When you start to do inferencing, there are things you can infer from the typed statement that you can't infer from the untyped statement.
jcoyne: BUt I agree that Fedora is not being helpful here.09:18
<escowles>ajs6f: i agree that plain literals and xsd:strings aren't fully equivalent
<ajs6f>jcoyne: Maybe this is ({sigh}) something that needs a little more configurability.
<escowles>imho, we should at least have an option to not throw an exception for trying to add/remove managed properites -- we could log them instead09:19
<ajs6f>Hm: http://www.w3.org/TR/rdf11-concepts/#dfn-literal-term-equality09:20
escowles: cbeer and Rob Sanderson went into this (what to do about managed predicates) pretty thoroughly in connexion with LDP requirements.
<jcoyne>Oh great.
<ajs6f>But I don't know what parameters of action eventuated.09:21
<escowles>ajs6f: yep, i agree that there is some difference (that "0" may mean something to someone), but those two values are equal in the Java .equals() sense, and i think you put it just right when you said we're not being helpful with that kind of difference09:22
<ajs6f>escowles: But I can think of use cases for wanting to be able to distinguish them, not so much for datatypes but for language tags/
<escowles>ajs6f: definitely, "fort"@fr and "fort"@en are different words09:23
<ajs6f>escowles: I think we need to be flexible here, to accommodate straightforward stuff like what jcoyne is trying to do.
<jcoyne>The spec seems vague in this particular case "Simple literals are syntactic sugar for abstract syntax http://www.w3.org/TR/rdf11-concepts/#dfn-literal with the http://www.w3.org/TR/rdf11-concepts/#dfn-datatype-irihttp://www.w3.org/TR/rdf11-concepts/#dfn-datatype-iri http://www.w3.org/2001/XMLSchema#string."
<ajs6f>jcoyne: That seems precise to me.
<jcoyne>"Simple literals are syntactic sugar for abstract syntax literals with the datatype IRI http://www.w3.org/2001/XMLSchema#string."09:24
<ajs6f>jcoyne: But what about the language tag.
<jcoyne>ajs6f: but does that contravene the value equality / literal equality statement09:25
?
<ajs6f>jcoyne: Simple literals don't define a lang tag, but equality requires that the two lang tags be equal.
* ajs6f leaves09:26
* ajs6f1 joins
<jcoyne>ajs6f: but in our case neither the datatyped string and the simple literal have lang tags09:27
s/and/nor/09:28
<ajs6f1>Right.09:29
<jcoyne>So, I assert that "nt:folder"^^<http://www.w3.org/2001/XMLSchema#string> and "nt:folder" are literally equivalent (because the string datatype is implied)
Fedora is currently saying, they are not.09:30
<ajs6f1>jcoyne: But equality is defined as character-by-character equality, which is not true here.
<jcoyne>"Simple literals are syntactic sugar for abstract syntax literals with the datatype IRI http://www.w3.org/2001/XMLSchema#string."09:31
<ajs6f1>jcoyone: I don't think the implied type works here for equality.
<jcoyne>So, after the implied datatype, then they are character-by-character equivalent
<ajs6f1>jcoyne: Right. And character-by-character is a syntatic quality.
jcoyne: And I understand the comparison to be made _before_ the implied type is available09:32
Here's another example:
do we want "1"^string and "1" to be equal? I don't.
I want "1" to be equal, if anything to, "1"^int, but that's a very hard inference to make.09:33
I think RDF is taking the safe but annoying way out here.
<jcoyne>Really?
<ajs6f1>Yep, really.
<escowles>i want "1"^string and "1" to be equal
<ajs6f1>Because XSD defines int < string.
escowles: That's fine. The point is that _we don't agree_.
<escowles>ajs6f1: sure, but i think my way should be the default ;)09:34
<jcoyne>ajs6f1: the point is, the spec doesn't make these cases clear.
<ajs6f1>So RDF says, "FIne! None of them are equal! And if you don't stop fighting I'll turn this car around!"
jcoyne: Well, no, I don't agree with you. I think the spec is clear, it's just not what you want. {grin}
<jcoyne>ajs6f1: escowles and I found two RDF implementations that don't agree.09:35
ajs6f1: So, I think it's demonstratable that the spec is unclear
<ajs6f1>jcoyne: No, it doesn't follow that the spec is ambiguous. One of those impls may simply be wrong.
<cbeer>ajs6f1, escowles : i hate the difference between jena and sesame.
<jcoyne>ajs6f1: And the wrong imple is sesame ;)09:36
<cbeer>ajs6f1: the other thing azaroth and I discussed (and maybe escowles and I?) was 1^int vs 1^double vs 1^othernumberictypes
<ajs6f1>jcoyne: Says you. {grin}
afk
<cbeer>ajs6f1: jena gives us https://jena.apache.org/documentation/javadoc/jena/com/hp/hpl/jena/rdf/model/Literal.html#sameValueAs(com.hp.hpl.jena.rdf.model.Literal)09:37
<jcoyne>Has anyone consulted with the editors of the RDF spec?
<cbeer>ajs6f1, escowles : the reason jcoyne's thing used to work is we used to blindly trust sesame to do the right thing with literal objects (which it never did for any numeric types), but now we use the Jena types everywhere: https://github.com/fcrepo4/fcrepo4/blob/master/fcrepo-http-commons/src/main/java/org/fcrepo/http/commons/responses/RdfStreamStreamingOutput.java#L18409:41
<escowles>cbeer: is there some property we can set in the RDF differencing process that would use sameValue for comparisons intead of equals?09:42
<cbeer>escowles: let me check. azaroth42 and I also discovered the Prefer: handling=lenient header yesterday, which might be useful.
<ajs6f1>all: some interesting points here:09:43
http://answers.semanticweb.com/questions/9781/regarding-plain-literal-and-rdfplainliteral-equality
<cbeer>escowles: oh, right.. the RDF differencing is caught up in a generic DifferencingIterator09:44
Set#contains
using Set#contains too.. so.. uh..
<escowles>cbeer: which is using .equals() obviously
<ajs6f1>cbeer: we could break a notion of "what does it mean to be equal" out of DifferencingIterator— it would just mean to stop using contains().09:45
Use a filter or some loops instead.
<escowles>ajs6f1: yes, that seems like a reasonable way to plug in something more flexible09:46
<ajs6f1>cbeer/escowles: I think a plain Comparator<Triple> would do fine there.
<jcoyne>ajs6f1: I've asked Richard Cyganiak to clarify the spec. I'll share the response if he writes back.09:47
<ajs6f1>cbeer/escowles: or a set with the appropriate notion of equality injected might work.
<pivotal-bot____>Chris Beer added comment: "I would strongly suggest not working on this until PR #542 is merged." https://www.pivotaltracker.com/story/show/8090388009:48
<cbeer>ajs6f1: i guess we should see what Model#difference does09:49
<ajs6f1>cbeer: Sure, but just watch out for anything that drags all the triiples into the heap.
<cbeer>ajs6f1: we're already dragging one of the sets into the heap
<ajs6f1>cbeer: The reason that I made that iterator machinery was to avoid that.
cbeer: Yeah, but it shouldn't be the triple of the resource (from the JCR).
triples.
<pivotal-bot____>Esme Cowles edited "Non-RDF resource versioning broken" https://www.pivotaltracker.com/story/show/8090388009:50
<cbeer>ajs6f1: don't think it needs to be.
<ajs6f1>cbeer: When we did drag everything into the heap, we had constant heap blowouts. fasseg went into that in detail.
<pivotal-bot____>Chris Beer added "Semantically equivalent triples should be equivalent for RDF diffing purposes." https://www.pivotaltracker.com/story/show/8090709409:51
Chris Beer started "Semantically equivalent triples should be equivalent for RDF diffing purposes." https://www.pivotaltracker.com/story/show/80907094
<jcoyne>cbeer++09:52
<pivotal-bot____>A. "Offratonix" Soroka edited "Triple equality should be pluggable for RDF diffing purposes." https://www.pivotaltracker.com/story/show/80907094
<cbeer>ajs6f1: pluggable?09:53
<ajs6f1>cbeer: Adjustable, or whatever you like. I don't want to use jcoyne's notion of equaity and he doesn't want to use mine. And I'm not at all sure that I understand what "semantically equivalent" means but I am sure that we don09:54
t agree about it. {grin}
* awead leaves09:56
<jcoyne>ajs6f1: here's an alternative suggestion that would get us to the same place: Don't put a String datatype on fcr:primaryType09:57
<ajs6f1>jcoyne: No, I'm far more worried about other kinds of triples.
For example,
foo hasParts "1"09:58
and
* awead joins
<ajs6f1>foo hasTimestamp "1"
<escowles>ajs6f1: for "semantic equality" i think untyped literals should be equivalent to ^^xsd:string, and "01"^^xsd:int should be equivalent to "1"^^xsd:int
<ajs6f1>Those two literals are not "semantically equivalent" to me. But maybe they are to you? Fine. we should both be able to work,
escoqles: That rally does contravene the spec.09:59
escowles: Unless you mean to define some Fedora-specific notion of "semantic equality"?
escowles: I'm not in favor of that. That's as much as to define canonical type maps into value spaces that aren't actually in the remit of the repository.10:00
<escowles>ajs6f1: we are introducing many new sites to RDF, and they will not know (or care) about the RDF spec's notion of equality
and to me it seems plainly at odds with what a naive notion of equality means10:01
<ajs6f1>escowles: They shouldn't be writing directly against an LDP instance. They should be using an appropriate framework like Hydra or Islandora or whatever.
escowles: Well, I don't agree with you there. I think syntatic equality is the most naive possible equality.
<jcoyne>ajs6f1: but the libraries those use, like RDF.rb and Jena are removing the string types
ajs6f1: so we need to patch every RDF library that does that?10:02
<escowles>ajs6f1: right, but this issue came up here because jcoyne's RDF-processing library treats "foo" and "foo"^^xsd:string as equivalent
and i had this same issue using jena
<ajs6f1>That's an error in that library, not a reason to create a non-portable idea of "semantic equality" and enforce it on the community.10:03
<escowles>and maybe i'm doing something wrong with jena to get this result (i think i convert <Triple> into <Statement> somewhere and probably screwed up the literal parsing or something
<ajs6f1>I don't undestand why it would be controversial to allow people to choose their notion of equality?
<escowles>ajs6f1: i'm fine with pluggable equality, for what it's worth10:04
<ajs6f1>It seems much harder to argue that everyone should have to use jcoyne's notion (or mine, or anyone else's).
Well, that's where we are with the ticket, so… patches welcome.
<escowles>ajs6f1: i assume cbeer is going to just fix this while we stand around arguing about it :)10:05
afk10:06
<ajs6f1>escowles: As long as he fixes it the way _I_ want! {grin}
escowles: I think you convert Triple to Statement either with:10:09
https://jena.apache.org/documentation/javadoc/jena/com/hp/hpl/jena/rdf/model/ResourceFactory.html#createStatement(com.hp.hpl.jena.rdf.model.Resource, com.hp.hpl.jena.rdf.model.Property, com.hp.hpl.jena.rdf.model.RDFNode)
or
https://jena.apache.org/documentation/javadoc/jena/com/hp/hpl/jena/rdf/model/Model.html#createStatement(com.hp.hpl.jena.rdf.model.Resource, com.hp.hpl.jena.rdf.model.Property, com.hp.hpl.jena.rdf.model.RDFNode)10:10
and in either case you shouldn't have to fool with the literal directly.
<jcoyne>ajs6f: Here's Richard's response: https://gist.github.com/jcoyne/a856be7cff78b2a9d71b10:15
"TL;DR In Turtle, "foo" and "foo"^^xsd:string mean *exactly the same* thing. Not just equivalent, but exactly the same. In the same way as true and "true"^^xsd:boolean are exactly the same thing."10:17
<ajs6f1> jcoyne: Okay, that does seem pretty unequivocal. I disagree with the idea, but it does seem to be the standard.10:18
<cbeer>ajs6f1: and it looks like Jena's Model#difference does the right thing.. now to just dig through the layer and find out what that is.
<jcoyne>ajs6f1: Good, because pluggable equivalence makes me shudder.10:19
<ajs6f1>jcoyne: I don't know why.
Being forced to use other peoples' notions of equivalence annoys me.
<jcoyne>ajs6f1: It just seems like one of those things that causes a bug 3 years down the road.10:20
You can't figure out why X works on machine A, but works differntly on machine B
<cbeer>ajs6f1: aw.. Jena has a protected addCommon method that takes an iterator.
<ajs6f1>jcoyne: Exactly, except I'm think of it the other way. The bug comes up in my data because software was interpreting two things as equal when I didn't mean for that to happen.
and I didn't know it was happenoing.
I'm still trying to figure out what:10:21
foo hasProp "true"
means
is a boolean? No. It's a string.
<jcoyne>I'm much happier with consistently wrong, than sometimes wrong
<ajs6f1>I'm much happier with right, because I declared what the proper behavior is.10:22
<jcoyne>well, yes, that is the best of them all.
* dwilcox leaves
<jcoyne>For now I'll settle for not getting 409 Conflict.
<ajs6f1>Well, we're going to do what's spec'd, so that's an end to it.
There's another reason I would want pluggable equivalence (eventually, not now), and escowles touched on it.10:23
What if I want to declare two things semantically equivalent because I know that two resources have sameAs between them.10:24
E.g.
what if I want the the repo to treat example.com and example.org the same, for purposes of auth, or update, or whatever.
That's a really simple and easy way to integrate inferencing into the repo.
* osmandin joins10:25
<ajs6f1>That's why I'm loath to use words like "semantically equivalent" unless we all agree that we will all never agree what they means.
<jcoyne>Well, I don't think pluggable == has been excluded, just not necessary to fix the issue I have.
<ajs6f1>jcoyne: No, I don't want to slow down a fix for you. I just don't want to fix your problem at the expense of future functionality.10:26
* scossu joins10:30
* awoods joins10:37
* jonroby joins10:40
<ajs6f1>cbeer: Are you actually working the literal equality ticket?10:42
<cbeer>ajs6f1: i am literally working on it.10:43
<ajs6f1>Badump-ching!
cbeer: Okay, nm.
<pivotal-bot____>Esme Cowles started "Non-RDF resource versioning broken" https://www.pivotaltracker.com/story/show/8090388010:47
<cbeer>escowles: did you see my note?
<escowles>cbeer: looking at it now -- i'll just retest this under your branch to start10:48
<cbeer>escowles: it's probably still broken in the same way... but you'll get to fix it with models instead
escowles: and FedoraBinary#enableVersioning should do the right thing.
* github-ff joins10:51
[fcrepo4] cbeer created jena-graph-containment (+1 new commit): http://git.io/E1XPdg
fcrepo4/jena-graph-containment 2ade9b8 Chris Beer: Use Jena's notion of graph containment instead of our strict equality
* github-ff leaves
<cbeer>ajs6f1: does https://github.com/fcrepo4/fcrepo4/commit/2ade9b83fb8f5f7c199566a5b56448c7db36cce8 look reasonable? just using graphs instead of sets
sorry, i guess the diff is a little nasty: https://github.com/fcrepo4/fcrepo4/blob/2ade9b83fb8f5f7c199566a5b56448c7db36cce8/fcrepo-kernel/src/main/java/org/fcrepo/kernel/utils/iterators/GraphDifferencingIterator.java10:52
(and I need to find out how to actually build a new graph the right way)
<ajs6f1>cbeer: I'll comment on that branch at github, k?10:53
<cbeer>ajs6f1: please.10:54
ajs6f1: and i was only going for "get that new test passing", not shippable.
* travis-ci joins
fcrepo4/fcrepo4#2769 (jena-graph-containment - 2ade9b8 : Chris Beer): The build has errored.
Change view : https://github.com/fcrepo4/fcrepo4/commit/2ade9b83fb8f
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38268793
* travis-ci leaves
<pivotal-bot____>Osman Din added comment: "https://github.com/osmandin/fcrepo4/commit/12cdab9dca9b84196383d80b70057437590fc35f" https://www.pivotaltracker.com/story/show/7946040411:04
* ajs6f1 leaves11:11
<pivotal-bot____>Esme Cowles added comment: "@awoods this ticket is accepted, but https://github.com/fcrepo4/fcrepo-message-consumer/pull/57 isn't merged." https://www.pivotaltracker.com/story/show/8056430211:12
* nikhiltri joins11:23
* dwilcox joins11:29
* nikhiltri leaves11:31
<pivotal-bot____>Andrew Woods added comment: "@escowles, looking at the history of this ticket, since it was a "chore", when you "finished" it, the ticket �" https://www.pivotaltracker.com/story/show/8056430211:33
Andrew Woods rejected "Fix JMS message consumer build" https://www.pivotaltracker.com/story/show/80564302
* nikhiltri joins11:39
* rtuckey joins11:41
<osmandin>cbeer: I was wondering if I could get your valuable input on https://github.com/osmandin/fcrepo4/blob/fcrversion/fcrepo-kernel-impl/src/test/java/org/fcrepo/integration/kernel/impl/FedoraResourceImplIT.java#L40111:42
cbeer: It seems the test (ignore the @Ignore) is creating the version label (v 0.0.1) on the root version, and only the root version is getting created
cbeer: whereas in UI, updating an object results in 2 versions
cbeer: basically, I'd like to create a non-root version in that test
<cbeer>osmandin: i know next to nothing about versioning. maybe escowles or MikeAtUVA can help11:43
<osmandin>cbeer: ok, thx
<cbeer>escowles: huh. so, in Jena, Graph#contains respects the RDF equivalence of typed and untypes resources, but Graph#delete does not.11:45
but Graph#remove(subject, predicate, object) works.. so I guess that's good11:46
* jmignault joins11:55
* github-ff joins
[fcrepo4] cbeer force-pushed jena-graph-containment from 2ade9b8 to b25d070: http://git.io/1Ow_rw
fcrepo4/jena-graph-containment b25d070 Chris Beer: Use Jena's notion of graph containment instead of our strict equality
* github-ff leaves
* github-ff joins11:58
[fcrepo4] cbeer force-pushed jena-graph-containment from b25d070 to 878b08e: http://git.io/1Ow_rw
fcrepo4/jena-graph-containment 878b08e Chris Beer: Use Jena's notion of graph containment instead of our strict equality
* github-ff leaves
<pivotal-bot____>Chris Beer finished "Triple equality should be pluggable for RDF diffing purposes." https://www.pivotaltracker.com/story/show/80907094
* github-ff joins
[fcrepo4] cbeer opened pull request #552: Use Jena's notion of graph containment instead of our strict equality (master...jena-graph-containment) http://git.io/Bmd6Pg
* github-ff leaves
* jmignault leaves12:00
<pivotal-bot____>Chris Beer added comment: "https://github.com/fcrepo4/fcrepo4/pull/552" https://www.pivotaltracker.com/story/show/8090709412:01
Chris Beer edited "Triple equality should use RDF equality for diffing purposes." https://www.pivotaltracker.com/story/show/8090709412:02
Chris Beer edited "Triple equality should use RDF equality for diffing purposes." https://www.pivotaltracker.com/story/show/80907094
* github-ff joins12:06
[fcrepo4] cbeer force-pushed jena-graph-containment from 878b08e to 9f3e4f0: http://git.io/1Ow_rw
fcrepo4/jena-graph-containment 9f3e4f0 Chris Beer: Use RDF equality when diff'ing user-provided triples against stored triples
* github-ff leaves
* github-ff joins12:07
[fcrepo4] cbeer force-pushed jena-graph-containment from 9f3e4f0 to 20530a4: http://git.io/1Ow_rw
fcrepo4/jena-graph-containment 20530a4 Chris Beer: Use RDF equality when diff'ing user-provided triples against stored triples
* github-ff leaves
* travis-ci joins
fcrepo4/fcrepo4#2773 (jena-graph-containment - 9f3e4f0 : Chris Beer): The build has errored.
Change view : https://github.com/fcrepo4/fcrepo4/compare/878b08ea9845...9f3e4f08e31e
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38276381
* travis-ci leaves
* travis-ci joins
fcrepo4/fcrepo4#2771 (jena-graph-containment - 878b08e : Chris Beer): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/b25d0703b3cf...878b08ea9845
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38275548
* travis-ci leaves
* travis-ci joins12:08
fcrepo4/fcrepo4#2770 (jena-graph-containment - b25d070 : Chris Beer): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/2ade9b83fb8f...b25d0703b3cf
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38275201
* travis-ci leaves
<cbeer>awoods: can you look at https://github.com/fcrepo4/fcrepo4/pull/552, so i can rebase ALL THE THINGS?
<awoods>cbeer: on a call
* github-ff joins12:11
[fcrepo4] osmandin opened pull request #553: Remove root version from triples (master...fcrversion-pr3) http://git.io/nER_vg
* github-ff leaves
<pivotal-bot____>Osman Din added comment: "https://github.com/fcrepo4/fcrepo4/pull/553" https://www.pivotaltracker.com/story/show/79460404
Osman Din finished "Filter out root version from RDF" https://www.pivotaltracker.com/story/show/79460404
* ajs6f joins12:17
cbeer: ping12:18
<cbeer>ajs6f: pong
* travis-ci joins
fcrepo4/fcrepo4#2775 (jena-graph-containment - 20530a4 : Chris Beer): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/9f3e4f08e31e...20530a466da0
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38276432
* travis-ci leaves
<ajs6f>cbeer: sorry about randomly disappearing— my neighborhood lost power (still don't know why), so I had to peddle downtown to a teahouse for wifi.
<cbeer>ajs6f: excuses, excuses. here's the PR: https://github.com/fcrepo4/fcrepo4/pull/55212:19
<ajs6f>cbeer: {grin} Thanks.
<cbeer>ajs6f: if you want to keep the generics, you'll have to show me how.
<ajs6f>cbeer: yeah, I realized on the way over that it wasn't obvious, but I think I can. Shall I just try to send a PR to yours?
<cbeer>ajs6f: or we get awoods to merge the thing and then iterate.12:20
* jonroby leaves
<ajs6f>cbeer: Good point. Let's do that. jcoyne is a dangerous man to cross.
<awoods>cbeer/ajs6f: back12:25
cbeer/ajs6f: where is the priority?
<ajs6f>awoods: ask jcoyne.12:26
<cbeer>awoods: #522.
<awoods>thanks12:27
<cbeer>awoods: sorry, #552
awoods: allowed by #542, but that'll conflict with #552.
s/allowed/followed/12:28
<awoods>cbeer: ok
* github-ff joins
[fcrepo4] cbeer created fedora-resource (+1 new commit): http://git.io/p_gCCQ
fcrepo4/fedora-resource 0f18e84 Chris Beer: Allow fedora:binary to receive all fedora:resource properties
* github-ff leaves
<ajs6f>cbeer: When you did Jersey2, did you have any trouble with MultiExceptions and the exception mappers? I think that's the root of the weird traces about which awoods complained.
<pivotal-bot____>Chris Beer added "Allow full range of RDF properties to be expressed on a fedora:binary node" https://www.pivotaltracker.com/story/show/80920870
Chris Beer started "Allow full range of RDF properties to be expressed on a fedora:binary node" https://www.pivotaltracker.com/story/show/80920870
Chris Beer added comment: "https://github.com/fcrepo4/fcrepo4/pull/554" https://www.pivotaltracker.com/story/show/8092087012:29
* github-ff joins
[fcrepo4] cbeer opened pull request #554: Allow fedora:binary to receive all fedora:resource properties (master...fedora-resource) http://git.io/t3x0Dw
* github-ff leaves
<pivotal-bot____>Chris Beer finished "Allow full range of RDF properties to be expressed on a fedora:binary node" https://www.pivotaltracker.com/story/show/80920870
* rtuckey leaves12:32
* github-ff joins12:34
[fcrepo4] cbeer force-pushed fedora-resource from 0f18e84 to 80d682b: http://git.io/InZ_XQ
fcrepo4/fedora-resource 80d682b Chris Beer: Allow fedora:binary to receive all fedora:resource properties
* github-ff leaves
* travis-ci joins12:35
fcrepo4/fcrepo4#2777 (fedora-resource - 0f18e84 : Chris Beer): The build has errored.
Change view : https://github.com/fcrepo4/fcrepo4/commit/0f18e84e68f4
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38278875
* travis-ci leaves
<pivotal-bot____>Chris Beer added "Replace hasContent/isContentOf in RDF with describes/isDescribedBy" https://www.pivotaltracker.com/story/show/8092139012:36
Chris Beer estimated "Replace hasContent/isContentOf in RDF with describes/isDescribedBy" as 0 points https://www.pivotaltracker.com/story/show/80921390
* travis-ci joins12:43
fcrepo4/fcrepo4#2779 (fedora-resource - 80d682b : Chris Beer): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/0f18e84e68f4...80d682b1637a
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38279357
* travis-ci leaves
<osmandin>afk a bit12:44
<awoods>cbeer/ajs6f: PR-552 looks good for now... I your two content (based on the github chatter)?
cbeer/ajs6f: PR-552 looks good for now... are you two content (based on the github chatter)?
<ajs6f>awoods: If cbeer is good, then I am. I have things to do there, but there's no point in waiting now.
<awoods>ajs6f: thanks
<cbeer>awoods: i am, and look forward to ajs6f's future enhancements.
* github-ff joins12:45
[fcrepo4] awoods pushed 2 new commits to master: http://git.io/HFqhCA
fcrepo4/master 20530a4 Chris Beer: Use RDF equality when diff'ing user-provided triples against stored triples
fcrepo4/master 019365e Andrew Woods: Merge pull request #552 from fcrepo4/jena-graph-containment...
* github-ff leaves
<ajs6f>cbeer: You mean like my new cybertronic ankles and semantic-inferencing-equipped knees?
<pivotal-bot____>Andrew Woods delivered "Triple equality should use RDF equality for diffing purposes." https://www.pivotaltracker.com/story/show/80907094
A. "Offratonix" Soroka edited "Triple equality should use RDF literal equality for diffing purposes." https://www.pivotaltracker.com/story/show/8090709412:46
<awoods>ajs6f/cbeer: fyi, yesterday's f4 training was a hit... the crowd was loving the intuitive LD interactions.
<ajs6f>awoods: That's the way… suck them in gently, then, once they've been lulled, _twist the knife_.12:47
* github-ff joins
[fcrepo4] cbeer force-pushed resource-converter from 13d4a6d to 80e382b: http://git.io/mU2H3A
fcrepo4/resource-converter affd8b0 Chris Beer: Implement a Converter<Resource,FedoraResource> to replace the Node converter
fcrepo4/resource-converter 80e382b Chris Beer: Use FedoraResource#getContainer/getChildren when serializating resources...
* github-ff leaves
* github-ff joins12:50
[fcrepo4] cbeer force-pushed resource-converter from 80e382b to 319285c: http://git.io/mU2H3A
fcrepo4/resource-converter 319285c Chris Beer: Use FedoraResource#getContainer/getChildren when serializating resources...
* github-ff leaves
* travis-ci joins12:51
fcrepo4/fcrepo4#2782 (resource-converter - 80e382b : Chris Beer): The build has errored.
Change view : https://github.com/fcrepo4/fcrepo4/compare/13d4a6dbc461...80e382b9f490
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38280548
* travis-ci leaves
<awoods>cbeer: PR-542[ERROR] COMPILATION ERROR :12:53
[INFO] -------------------------------------------------------------
[ERROR] /home/awoods/programming/java/apps/fedora/futures/fcrepo4/fcrepo-kernel-impl/src/main/java/org/fcrepo/kernel/impl/FedoraResourceImpl.java:[144,32] cannot find symbol
symbol: method singletonIterator(org.fcrepo.kernel.FedoraResource)
<cbeer>awoods: see 319285c
<awoods>cbeer: I apparently just missed your force-push12:55
* awead leaves
<awoods>cbeer/ajs6f: are we using DefaultIdentifierTranslator.java anywhere now?12:56
<cbeer>awoods: tests.12:57
<ajs6f>awoods: It's hard to imagine someone using it in deployment.
<cbeer>ajs6f, awoods : and we actively discourage it.
<ajs6f>cbeer: Right, it exposes internal ids, right?12:58
<cbeer>ajs6f: not any more, but nearly so.
<awoods>cbeer: why do we need it in tests? If we do indeed need it, then we should move it under the test/java directory, no?
<ajs6f>cbeer;/awoods: Might be worth moving to a test package, renaming it, and commenting it to explain why it shouldn't be used for anything but tests.
<awoods>ajs6f: +1
* travis-ci joins
fcrepo4/fcrepo4#2784 (resource-converter - 319285c : Chris Beer): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/80e382b9f490...319285c646e2
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38280778
* travis-ci leaves
<cbeer>awoods: i'd also suggest that's a separate concern than #542. it's always been used only in tests.12:59
<ajs6f>awoods: And then putting a big sign on it.
<awoods>cbeer: agreed, it just jumped out
<pivotal-bot____>Andrew Woods added "Move DefaultIdentifierTranslator.java to tests" https://www.pivotaltracker.com/story/show/8092314013:00
Andrew Woods edited "Move DefaultIdentifierTranslator.java to tests" https://www.pivotaltracker.com/story/show/80923140
<awoods>cbeer: three times in a row I have gotten the following failure when building PR-54213:05
Tests run: 9, Failures: 1, Errors: 0, Skipped: 4, Time elapsed: 17.867 sec <<< FAILURE! - in org.fcrepo.integration.FedoraHtmlResponsesIT
testCreateNewNodeWithGeneratedId(org.fcrepo.integration.FedoraHtmlResponsesIT) Time elapsed: 2.548 sec <<< FAILURE!
java.lang.AssertionError: Didn't see new information in page!
at org.junit.Assert.fail(Assert.java:88)
at org.junit.Assert.assertTrue(Assert.java:41)
at org.fcrepo.integration.FedoraHtmlResponsesIT.testCreateNewNodeWithGeneratedId(FedoraHtmlResponsesIT.java:131)
* rtuckey joins
<awoods>cbeer: I need to return my rental car... bb-soon
<ajs6f>awoods: It's okay, I'll do that for you.13:06
<cbeer>awoods: i can take a look, but i doubt it's related.
but htmlunit is also impossible to debug.
<escowles>cbeer: how do you create a node with a prefixed name now? the message-consumer ITs are failing now that they can't register the namespaces with fcr:namespaces any more...13:11
<cbeer>escowles: using a feature yet-to-be-developed
<escowles>oh, i see...
<cbeer>escowles: or pre-registering it in the CND
<ajs6f>escowles: Either you just use the prefix (which is now opaque)
escowles: Or you do the CND fandango.13:12
<cbeer>or creating a property using a namespace-aware RDF flavor with that namespace.
<ajs6f>cbeer: but that won't create a resource, will it?
<cbeer>ajs6f: no. escowles would have to add the property to some node (and then remove it, for all we care)13:13
and then do his business.
<escowles>ajs6f: for the test, i'd be happy to just use the namespace, but even URL-escaped it's not working for me
<ajs6f>Wait, you're saying that if you've declared the prefix in the RDF serialization, it magically gets created as a namespace?
<escowles>e.g., various forms of PUT http://localhost:8080/rest/fedora:system/fedora:transform/fedora:ldpath/dc/http%3A//fedora.info/definitions/v4/indexingtest#book
<cbeer>ajs6f: yes. JCR uses the same namespaces for properties and nodes.13:14
<ajs6f>escowles: What's coming back for that request?
<escowles>400: The namespace prefix (http) has not been registered
<ajs6f>cbeer: I understand why we do that, but I dunno, it seems a little weird to me.
<escowles>urlencoding the colon doesn't help...
<ajs6f>Declaring a prefix in RDF (which is not supposed to alter the semantics of the content) actually alters the repository (even if in a minor and helpful way).13:15
escowles: Did you try just:
http://localhost:8080/rest/fedora:system/fedora:transform/fedora:ldpath/dc/indexingtest:book
without bothering to explain what you mean by "indexingtest:"?13:16
<escowles>ajs6f: yep, that's where i started: The namespace prefix (indexingtest) has not been registered
<ajs6f>escowles: Have you tried using an "enhanced" CND?
* awead joins
<escowles>ajs6f: i think i'll go that route, since that seems like a better pattern than using RDF updates to register namespaces...13:17
<ajs6f>escowles: Agreed. As cbeer says, there s a feature yet-to-be-developed here.13:18
cbeer/awoods: I almost think it would have been better to leave the namespace update stuff in, even if it uses CND, and iterate on it.13:19
<pivotal-bot____>Esme Cowles started "Fix JMS message consumer build" https://www.pivotaltracker.com/story/show/8056430213:24
* nikhiltri leaves13:28
<cbeer>awoods: ah, i see it now, ok.13:34
* github-ff joins13:39
[fcrepo4] ajs6f created SLF4JLoggingForJersey (+1 new commit): http://git.io/bmre4Q
fcrepo4/SLF4JLoggingForJersey 9ff5e5a ajs6f: Adjusting logging for Jersey to use SLF4J
* github-ff leaves
<pivotal-bot____>A. "Offratonix" Soroka added comment: "This was a problem where Jersey wasn't properly connected to SLF4J, although it raises some questio�" https://www.pivotaltracker.com/story/show/8068151813:40
A. "Offratonix" Soroka added comment: "https://github.com/fcrepo4/fcrepo4/pull/555" https://www.pivotaltracker.com/story/show/8068151813:41
* github-ff joins
[fcrepo4] ajs6f opened pull request #555: Adjusting logging for Jersey to use SLF4J (master...SLF4JLoggingForJersey) http://git.io/UdhcjQ
* github-ff leaves
<pivotal-bot____>A. "Offratonix" Soroka finished "Dependency errors in fcrepo-http-api tests" https://www.pivotaltracker.com/story/show/80681518
* github-ff joins13:42
[fcrepo-message-consumer] escowles pushed 1 new commit to literal-dc-subject: http://git.io/VNR1ig
fcrepo-message-consumer/literal-dc-subject b54da71 Esmé Cowles: Using CND file to specify namespace instead of (now-removed) fcr:nodeyptes rest api
* github-ff leaves
<pivotal-bot____>Esme Cowles added comment: "I've updated #57 to work around the removal of the fcr:namespaces rest api: ""13:43
https://github.com/fcrepo4/fcrepo�" https://www.pivotaltracker.com/story/show/80564302
Esme Cowles finished "Fix JMS message consumer build" https://www.pivotaltracker.com/story/show/8056430213:44
* travis-ci joins13:47
fcrepo4/fcrepo4#2786 (SLF4JLoggingForJersey - 9ff5e5a : ajs6f): The build has errored.
Change view : https://github.com/fcrepo4/fcrepo4/commit/9ff5e5a6940c
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38285170
* travis-ci leaves
<pivotal-bot____>A. "Offratonix" Soroka added "Add logging back in for integration tests" https://www.pivotaltracker.com/story/show/8092673413:48
<ajs6f>Damn you, checkstyle.
* github-ff joins13:50
[fcrepo4] ajs6f pushed 1 new commit to SLF4JLoggingForJersey: http://git.io/YY2e0w
fcrepo4/SLF4JLoggingForJersey f7f4fab ajs6f: Jiggling for checkstyle's prurient delight
* github-ff leaves
<ajs6f>awoods: ping13:55
* travis-ci joins13:59
fcrepo4/fcrepo4#2788 (SLF4JLoggingForJersey - f7f4fab : ajs6f): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/9ff5e5a6940c...f7f4fab7edfb
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38286212
* travis-ci leaves
* github-ff joins
[fcrepo4] cbeer force-pushed resource-converter from 319285c to 6cc2ad4: http://git.io/mU2H3A
fcrepo4/resource-converter 6cc2ad4 Chris Beer: Use FedoraResource#getContainer/getChildren when serializating resources...
* github-ff leaves
* ajs6f leaves14:04
* ajs6f joins
cbeer: I just noticed that there's an AbstractResource for JAX-RS controllers and a FedoraBaseResource, and all actual controllers inherit from FedoraBaseResource. Is there a reason we broke those apart?14:05
(And FedoraBaseResource < AbstractResource)
<cbeer>ajs6f: AbstractResource is in http.commons14:06
FedoraBaseResource is in http.api
<ajs6f>Ok. ?
<cbeer>ajs6f: and translator() needs to have access to FedoraLdp.class
and setUpJMSBaseURIs might only be a http api concern. i'm not sure.
ajs6f: or, it isn't that we broke AbstractResource apart.. we pushed common stuff from .http.api down.14:07
<ajs6f>Ok, leave setUpJMSBaseURIs alone. Seems a little odd that a field like translator() needs access to a different controller class.
Is FedoraLdp doing double duty as a controller and some kind of helper?14:08
<cbeer>ajs6f: translator() needs to know what class is serving up the resources we're talking about.
* travis-ci joins
fcrepo4/fcrepo4#2790 (resource-converter - 6cc2ad4 : Chris Beer): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/319285c646e2...6cc2ad4a3234
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38286892
* travis-ci leaves
<ajs6f>cbeer: Fair enough, but is that determination in FedoraLdp?
<cbeer>ajs6f: yes? that's what we're saying resolves our URIs in http-api14:09
so it's JAX-RS @Path information is important.
<ajs6f>cbeer: Okay, just checking. Sounds to me like at some point, we want to factor that logic out of FedoraLdp (which is a controller in its own right) and into some specific type.14:10
cbeer: I'm not trying to make a big deal out of this. Just trying to understand our design.
<cbeer>ajs6f: yep, could do. HttpResourceConverter just needs a UriBuilder14:11
<ajs6f>cbeer: Yeah, something like that. Well, another one for after relase, for sure.
<cbeer>ajs6f: and there's a convenient method that all our API should be using to get it14:12
<ajs6f>cbeer: Yeah, it's a more-or-less universal concern for HTTP-transacting code.
It should be easy and clear to deal with.
<pivotal-bot____>A. "Offratonix" Soroka added "Factor path-connected translation logic out of FedoraLdp" https://www.pivotaltracker.com/story/show/8092888614:16
* github-ff joins14:20
[fcrepo4] ajs6f force-pushed SLF4JLoggingForJersey from f7f4fab to 32d456b: http://git.io/rBNbww
fcrepo4/SLF4JLoggingForJersey fb7dd21 ajs6f: Adjusting logging for Jersey to use SLF4J
fcrepo4/SLF4JLoggingForJersey 32d456b ajs6f: Jiggling for checkstyle's prurient delight
* github-ff leaves
<ajs6f>cbeer: Am I right in thinking that the change to use Jena's Graph type left us using Jena's notion of RDF equality?14:27
<cbeer>ajs6f: no. the change to Graph just let us drop the Quads we weren't using.14:28
<pivotal-bot____>Osman Din started "Children not rendering in UI" https://www.pivotaltracker.com/story/show/80575150
<ajs6f>cbeer: Wait, so we're _not_ using Jena's notion of RDF_equality?
<cbeer>ajs6f: as of https://github.com/fcrepo4/fcrepo4/commit/20530a466da033b3414b2aae5e5f47be5600927d, we are.
<ajs6f>cbeer: Even tho' we're using Graph.contains()?
<cbeer>but it has nothing to do with Dataset => Graph.14:29
<ajs6f>cbeer: Fair enough— just wanted to verify where we are getting equality from.
bbs
<cbeer>ajs6f: oh, unless you were asking the change from Set<> to Graph is why we have Jena's equality.. then, yes, yes it is.
* ajs6f leaves14:30
* ajs6f joins
* travis-ci joins
fcrepo4/fcrepo4#2792 (SLF4JLoggingForJersey - 32d456b : ajs6f): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/f7f4fab7edfb...32d456b263d3
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38289446
* travis-ci leaves
* github-ff joins14:34
[fcrepo4] osmandin opened pull request #556: Elevate ldp:contains for visibility in UI (master...80575150-1) http://git.io/hOGc0g
* github-ff leaves
* github-ff joins14:36
[fcrepo4] cbeer force-pushed tombstones from dd471e3 to c48d68e: http://git.io/UDxzXw
fcrepo4/tombstones c48d68e Chris Beer: Add Tombstone resource when a resource is deleted, preventing the re-use of URIs until the tombstone is also removed
* github-ff leaves
* github-ff joins
[fcrepo4] cbeer force-pushed prefer-server-managed from b31fb64 to 333a2f7: http://git.io/t-pmHg
fcrepo4/prefer-server-managed 2e0d5f7 Chris Beer: Move dc:describable into the fedora namespace
fcrepo4/prefer-server-managed b2ec461 Chris Beer: Add support for ServerManaged preference
fcrepo4/prefer-server-managed 07e2081 Chris Beer: Apply server-managed preferences to getResourceTriples()
* github-ff leaves
* nikhiltri joins14:38
* github-ff joins
[fcrepo4] cbeer closed pull request #543: Use FedoraResource#getContainer/getChildren when serializating resources (resource-converter...skip-pairtree-nodes) http://git.io/ad06Kg
* github-ff leaves
* github-ff joins14:40
[fcrepo4] osmandin closed pull request #556: Elevate ldp:contains for visibility in UI (master...80575150-1) http://git.io/hOGc0g
* github-ff leaves
* travis-ci joins14:48
fcrepo4/fcrepo4#2796 (prefer-server-managed - 333a2f7 : Chris Beer): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/b31fb64c604b...333a2f7c8027
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38290875
* travis-ci leaves
* travis-ci joins
fcrepo4/fcrepo4#2795 (tombstones - c48d68e : Chris Beer): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/dd471e3bae03...c48d68ea1b2e
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38290836
* travis-ci leaves
<ajs6f>cbeer/jcoyne: Jena doesn't impl RDF literal equals correctly, either.14:50
* github-ff joins14:51
[fcrepo4] osmandin opened pull request #557: Elevate ldp:contains for visibility in UI (master...80575150-1) http://git.io/zW81Kw
* github-ff leaves
<ajs6f>It elides the type issue by ignoring it, but it also ignores the lang tag.
<cbeer>ajs6f: good thing fcrepo4 doesn't handle lang tags
<pivotal-bot____>Osman Din added comment: "https://github.com/fcrepo4/fcrepo4/pull/557" https://www.pivotaltracker.com/story/show/80575150
Osman Din finished "Children not rendering in UI" https://www.pivotaltracker.com/story/show/80575150
<ajs6f>cbeer: We don't? Seriously, I have use cases here
<jcoyne>ajs6f: RDF is too hard. Let's use XML.
<cbeer>ajs6f: JCR only gives us a type system, not a lang systme.
<ajs6f>jcoyne: Let's use cuniform.
cbeer: Right. &(^(^(&14:52
<cbeer>ajs6f: although i've been thinking about implementing langs as property name suffixes somehow.
but we haven't had that use case yet
<ajs6f>cbeer: Well, we do, but it's a ways off for now.
We do from UVa, that is.
And I haven't written it because we haven't gotten that far.14:53
cbeer: Okay, I think the thing here is for me to send a PR to Jena.14:54
Wow. Jena is big.14:58
<cbeer>understatement.14:59
<ajs6f>Aye. If there's one thing at which I excel beyond the wildest dreams of the most ambitious men, it's understatement.
cbeer: Now that I think about it, RDF's lang system is essentially a demand for dependent types.15:01
* escowles leaves15:14
<ajs6f>out for the day15:16
* ajs6f leaves
* github-ff joins15:21
[fcrepo-module-auth-xacml] osmandin opened pull request #26: Fix compile and test errors (master...failedpr) http://git.io/kdKaUA
* github-ff leaves
<pivotal-bot____>Osman Din added comment: "Fake PR: https://github.com/fcrepo4/fcrepo-module-auth-xacml/pull/26" https://www.pivotaltracker.com/story/show/8056807615:22
Osman Din finished "Maintain fcrepo-module-auth-rbacl/xacml build" https://www.pivotaltracker.com/story/show/80568076
* github-ff joins15:24
[fcrepo4] cbeer pushed 1 new commit to prefer-server-managed: http://git.io/WhHD_g
fcrepo4/prefer-server-managed 65d325a Chris Beer: Add tests for PreferTag
* github-ff leaves
<osmandin>afk15:27
* travis-ci joins15:32
fcrepo4/fcrepo4#2799 (prefer-server-managed - 65d325a : Chris Beer): The build has errored.
Change view : https://github.com/fcrepo4/fcrepo4/compare/333a2f7c8027...65d325a3aef3
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38296649
* travis-ci leaves
<awoods>cbeer: thoughts: https://github.com/fcrepo4/fcrepo4/pull/542/files#r19037724
cbeer: otherwise, it is ready to go.
<cbeer>awoods: yep, grander plans. we have a couple places were we should use something like that.15:33
* github-ff joins15:35
[fcrepo4] awoods closed pull request #542: Implement a Converter<Resource,FedoraResource> to replace the Node converter (master...resource-converter) http://git.io/5LWd5Q
* github-ff leaves
<pivotal-bot____>Andrew Woods delivered "Update our IdentifierConverter to convert between RDF Resources and FedoraResources" https://www.pivotaltracker.com/story/show/80789814
* github-ff joins15:36
[fcrepo4] cbeer force-pushed prefer-server-managed from 65d325a to c541025: http://git.io/t-pmHg
fcrepo4/prefer-server-managed 81bd220 Chris Beer: Allow clients to send Prefer header to control what properties they intend to operate against...
fcrepo4/prefer-server-managed c541025 Chris Beer: Add tests for PreferTag
* github-ff leaves
* dwilcox leaves15:38
* travis-ci joins15:44
fcrepo4/fcrepo4#2802 (prefer-server-managed - c541025 : Chris Beer): The build has errored.
Change view : https://github.com/fcrepo4/fcrepo4/compare/65d325a3aef3...c5410259e956
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38297532
* travis-ci leaves
* github-ff joins15:45
[fcrepo4] cbeer force-pushed prefer-server-managed from c541025 to 5b058c7: http://git.io/t-pmHg
fcrepo4/prefer-server-managed ee59fee Chris Beer: Allow clients to send Prefer header to control what properties they intend to operate against...
fcrepo4/prefer-server-managed 5b058c7 Chris Beer: Add tests for PreferTag
* github-ff leaves
* travis-ci joins15:46
fcrepo4/fcrepo4#2801 (master - daf10ec : Andrew Woods): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/019365e504d1...daf10ece4b5d
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38297464
* travis-ci leaves
* github-ff joins15:47
[fcrepo4] cbeer force-pushed prefer-server-managed from 5b058c7 to 4fddadb: http://git.io/t-pmHg
fcrepo4/prefer-server-managed 4fddadb Chris Beer: Add tests for PreferTag
* github-ff leaves
* github-ff joins15:49
[fcrepo4] cbeer force-pushed prefer-server-managed from 4fddadb to 2009fd6: http://git.io/t-pmHg
fcrepo4/prefer-server-managed 2009fd6 Chris Beer: Add tests for PreferTag
* github-ff leaves
* github-ff joins15:50
[fcrepo4] awoods pushed 1 new commit to master: http://git.io/S7VHFQ
fcrepo4/master 58bdaa2 ajs6f: Adjust logging for Jersey to use SLF4J...
* github-ff leaves
<pivotal-bot____>Andrew Woods delivered "Dependency errors in fcrepo-http-api tests" https://www.pivotaltracker.com/story/show/8068151815:51
* github-ff joins
[fcrepo4] awoods deleted SLF4JLoggingForJersey at 32d456b: http://git.io/Ppe1NQ
* github-ff leaves
* travis-ci joins
fcrepo4/fcrepo4#2804 (prefer-server-managed - 5b058c7 : Chris Beer): The build has errored.
Change view : https://github.com/fcrepo4/fcrepo4/compare/c5410259e956...5b058c7be3ed
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38298260
* travis-ci leaves
<pivotal-bot____>Andrew Woods edited "getChildren() on FedoraResource" https://www.pivotaltracker.com/story/show/71611298
<cbeer>grumble grumble checkstyle grumble grumble
* github-ff joins
[fcrepo4] cbeer force-pushed prefer-server-managed from 2009fd6 to 77f7769: http://git.io/t-pmHg
fcrepo4/prefer-server-managed 77f7769 Chris Beer: Add tests for PreferTag and LdpPreferTag
* github-ff leaves
<pivotal-bot____>Chris Beer finished "Add preference for telling update operations what triples were received (via Prefer) when updating a resource" https://www.pivotaltracker.com/story/show/8087418415:53
Chris Beer added comment: "https://github.com/fcrepo4/fcrepo4/pull/549" https://www.pivotaltracker.com/story/show/80874184
Chris Beer added comment: "https://github.com/fcrepo4/fcrepo4/pull/549" https://www.pivotaltracker.com/story/show/80874156
Chris Beer finished "Add preference for excluding server managed properties when requesting a resource" https://www.pivotaltracker.com/story/show/80874156
* travis-ci joins15:55
fcrepo4/fcrepo4#2806 (prefer-server-managed - 4fddadb : Chris Beer): The build has errored.
Change view : https://github.com/fcrepo4/fcrepo4/compare/5b058c7be3ed...4fddadb2a5fb
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38298643
* travis-ci leaves
* travis-ci joins15:56
fcrepo4/fcrepo4#2808 (prefer-server-managed - 2009fd6 : Chris Beer): The build has errored.
Change view : https://github.com/fcrepo4/fcrepo4/compare/4fddadb2a5fb...2009fd61de75
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38298833
* travis-ci leaves
<pivotal-bot____>Andrew Woods added comment: "Now when I try to create a datastream in the HTML view, I get the following error: ""15:58
=========
Error creating d�" https://www.pivotaltracker.com/story/show/80744372
Andrew Woods rejected "Adding a datastream results in subsequent download" https://www.pivotaltracker.com/story/show/80744372
* travis-ci joins15:59
fcrepo4/fcrepo4#2809 (master - 58bdaa2 : ajs6f): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/daf10ece4b5d...58bdaa218af0
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38298983
* travis-ci leaves
<pivotal-bot____>Andrew Woods added comment: "Pending code review discussion." https://www.pivotaltracker.com/story/show/8056430216:00
Andrew Woods rejected "Fix JMS message consumer build" https://www.pivotaltracker.com/story/show/80564302
* travis-ci joins16:01
fcrepo4/fcrepo4#2810 (prefer-server-managed - 77f7769 : Chris Beer): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/2009fd61de75...77f7769d89b8
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38299080
* travis-ci leaves
<pivotal-bot____>Andrew Woods added comment: "This PR has already been closed, but is unmerged. What are the plans for addressesing the topic of this ticke�" https://www.pivotaltracker.com/story/show/8083722216:02
Andrew Woods rejected "Skip pairtree nodes when serializing a container to RDF" https://www.pivotaltracker.com/story/show/80837222
Andrew Woods added comment: "It looks like commons-httpclient-3.1 needs to be fixed along with this PR.16:27
[WARNING] The dependency commons-�" https://www.pivotaltracker.com/story/show/72875022
Andrew Woods rejected "Add enforce-victims-rule plugin (?)" https://www.pivotaltracker.com/story/show/72875022
* github-ff joins16:28
[fcrepo4] awoods pushed 1 new commit to master: http://git.io/qFWQ8g
fcrepo4/master 1021593 Andrew Woods: Merge pull request #546 from fcrepo4/other-resources...
* github-ff leaves
<pivotal-bot____>Andrew Woods delivered "Remove or Rename "Inlined Resources"" https://www.pivotaltracker.com/story/show/80736294
<osmandin>afk16:29
<pivotal-bot____>Andrew Woods added comment: "Pending response from #80837222" https://www.pivotaltracker.com/story/show/7161129816:31
Andrew Woods rejected "getChildren() on FedoraResource" https://www.pivotaltracker.com/story/show/71611298
* osmandin leaves16:34
<pivotal-bot____>Andrew Woods added comment: "Failed tests:16:40
FedoraNodesIT.testCopyToProjection:209 expected:<201> but was:<409>
FedoraNodesIT.testCopy�" https://www.pivotaltracker.com/story/show/80920870
Andrew Woods rejected "Allow full range of RDF properties to be expressed on a fedora:binary node" https://www.pivotaltracker.com/story/show/80920870
* travis-ci joins
fcrepo4/fcrepo4#2812 (master - 1021593 : Andrew Woods): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/58bdaa218af0...1021593779ad
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38302627
* travis-ci leaves
<pivotal-bot____>Chris Beer added comment: "The commit was part of https://github.com/fcrepo4/fcrepo4/pull/542." https://www.pivotaltracker.com/story/show/80837222
Chris Beer delivered "Skip pairtree nodes when serializing a container to RDF" https://www.pivotaltracker.com/story/show/80837222
Chris Beer started "Allow full range of RDF properties to be expressed on a fedora:binary node" https://www.pivotaltracker.com/story/show/8092087016:41
* github-ff joins16:43
[fcrepo4] cbeer force-pushed tombstones from c48d68e to b653d32: http://git.io/UDxzXw
fcrepo4/tombstones b653d32 Chris Beer: Add Tombstone resource when a resource is deleted, preventing the re-use of URIs until the tombstone is also removed
* github-ff leaves
<pivotal-bot____>Chris Beer finished "Allow full range of RDF properties to be expressed on a fedora:binary node" https://www.pivotaltracker.com/story/show/80920870
* github-ff joins
[fcrepo4] cbeer force-pushed fedora-resource from 80d682b to dbf3053: http://git.io/InZ_XQ
fcrepo4/fedora-resource dbf3053 Chris Beer: Allow fedora:binary to receive all fedora:resource properties
* github-ff leaves
<pivotal-bot____>Chris Beer added comment: "Yes, which is completely impossible. " https://www.pivotaltracker.com/story/show/7287502216:44
Andrew Woods added comment: "Suggestion?" https://www.pivotaltracker.com/story/show/7287502216:45
Chris Beer started "getChildren() on FedoraResource" https://www.pivotaltracker.com/story/show/71611298
Chris Beer finished "getChildren() on FedoraResource" https://www.pivotaltracker.com/story/show/71611298
Chris Beer added comment: "Doesn't matter to me. Depends how you feel about the possibility of insecure dependencies. I don't think anyone�" https://www.pivotaltracker.com/story/show/7287502216:46
Chris Beer finished "Add "tombstones" when an object is deleted" https://www.pivotaltracker.com/story/show/8085321216:47
Chris Beer added comment: "https://github.com/fcrepo4/fcrepo4/pull/548" https://www.pivotaltracker.com/story/show/80853212
Chris Beer started "Adding a datastream results in subsequent download" https://www.pivotaltracker.com/story/show/8074437216:51
Chris Beer finished "Adding a datastream results in subsequent download" https://www.pivotaltracker.com/story/show/80744372
Chris Beer added comment: "I'm unable to reproduce this. You'll have to provide additional logging or something." https://www.pivotaltracker.com/story/show/80744372
* rtuckey leaves
* travis-ci joins16:52
fcrepo4/fcrepo4#2815 (fedora-resource - dbf3053 : Chris Beer): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/80d682b1637a...dbf305361fa9
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38303957
* travis-ci leaves
* travis-ci joins16:53
fcrepo4/fcrepo4#2813 (tombstones - b653d32 : Chris Beer): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/c48d68ea1b2e...b653d3278655
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38303865
* travis-ci leaves
<pivotal-bot____>Andrew Woods delivered "Allow full range of RDF properties to be expressed on a fedora:binary node" https://www.pivotaltracker.com/story/show/80920870
* github-ff joins
[fcrepo4] awoods closed pull request #554: Allow fedora:binary to receive all fedora:resource properties (master...fedora-resource) http://git.io/t3x0Dw
* github-ff leaves
<awoods>cbeer: ping16:59
<cbeer>awoods: pong
<awoods>cbeer: re: https://github.com/fcrepo4/fcrepo4/pull/547
<cbeer>awoods: yep. i can't recreate your problem.
<awoods>cbeer: starting from a fresh f4, I am not getting the error mentioned in the ticket...
<cbeer>awoods: ok, i guess thats.. good? i don't know what state you were in before17:00
<awoods>cbeer: however, when I click on "Add" there is now no indication that anything has happened
<cbeer>awoods: javascript error?
<awoods>cbeer: I would expect to be redirected to /datastream/fcr:metadata
<cbeer>awoods: it should redirect you to fcr:metadata
<awoods>cbeer: I am not seeing any errors17:01
<cbeer>awoods: you'll have to debug that too, because i don't have that kind of error either.
<awoods>cbeer: just no response... but the datastream is actually created
cbeer: you get the successful redirect?
<cbeer>awoods: yes.
<awoods>cbeer: hmm, I definitely am not.17:02
* travis-ci joins17:03
fcrepo4/fcrepo4#2817 (master - 91c464d : Andrew Woods): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/1021593779ad...91c464d067ef
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38304857
* travis-ci leaves
<awoods>cbeer: steps: 1-start empty f4, 2-create object "collection", 3-as child of collection, create datastream "image", 4-see no indication that anything happened.
<cbeer>awoods: ok, i do see a javascript error when giving the datastream a name.17:06
<awoods>cbeer: does the error surface to the HTML view, or do you only see it in a js console?17:07
<cbeer>awoods: js console. i don't think we've ever surfaced javascript error in any other way, have we?
<awoods>cbeer: not that I know of
<cbeer>awoods: give that a try.17:08
* github-ff joins
[fcrepo4] cbeer force-pushed html-binary-redirect from 44dbb13 to 63fb6a9: http://git.io/SHNvSA
fcrepo4/html-binary-redirect 63fb6a9 Chris Beer: When creating new binary resources in the HTML UI, redirect to the datastream description after creating.
* github-ff leaves
* escowles joins17:13
* jcoyne leaves17:14
<awoods>cbeer: did you try it? using the exact same steps as above, I now get the image download when I click "Add"17:16
<cbeer>awoods: no. in the middle of something else. i guess we'll need to make sure the right describedby header is always present.17:17
* travis-ci joins17:21
fcrepo4/fcrepo4#2818 (html-binary-redirect - 63fb6a9 : Chris Beer): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/44dbb1330cc0...63fb6a9c2bb7
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38306611
* travis-ci leaves
<pivotal-bot____>Andrew Woods added comment: "<awoods> cbeer: steps: 1-start empty f4, 2-create object "collection", 3-as child of collection, create datas�" https://www.pivotaltracker.com/story/show/8074437217:26
Andrew Woods rejected "Adding a datastream results in subsequent download" https://www.pivotaltracker.com/story/show/8074437217:27
* acoburn joins17:33
* github-ff joins
[fcrepo-message-consumer] awoods pushed 1 new commit to master: http://git.io/rzPtHw
fcrepo-message-consumer/master 791d6d6 Esmé Cowles: Update dc:subject examples to use literals instead of URIs as values...
* github-ff leaves
* github-ff joins
[fcrepo-message-consumer] awoods deleted literal-dc-subject at b54da71: http://git.io/irtbng
* github-ff leaves
<pivotal-bot____>Andrew Woods added comment: "Resolved with: https://github.com/fcrepo4/fcrepo-message-consumer/commit/791d6d6e8e6086cb4fd069988985085e645a�" https://www.pivotaltracker.com/story/show/80564302
Andrew Woods delivered "Fix JMS message consumer build" https://www.pivotaltracker.com/story/show/80564302
Chris Beer started "Adding a datastream results in subsequent download" https://www.pivotaltracker.com/story/show/8074437217:41
* acoburn1 joins17:44
* acoburn leaves17:47
<pivotal-bot____>Chris Beer finished "Adding a datastream results in subsequent download" https://www.pivotaltracker.com/story/show/8074437217:54
* github-ff joins
[fcrepo4] cbeer force-pushed html-binary-redirect from 63fb6a9 to c1e0846: http://git.io/SHNvSA
fcrepo4/html-binary-redirect 5679928 Chris Beer: Return Link[rel=describes/describedby] headers for all requests that create a LDP-NR
fcrepo4/html-binary-redirect c1e0846 Chris Beer: When creating new binary resources in the HTML UI, redirect to the datastream description after creating.
* github-ff leaves
* github-ff joins17:57
[fcrepo4] awoods pushed 1 new commit to master: http://git.io/pyvwPg
fcrepo4/master 2a18f12 Chris Beer: Add enforce-victims-rule to prevent shipping with vulnerable dependencies...
* github-ff leaves
* github-ff joins
[fcrepo4] awoods opened pull request #558: Exclude httpclient from fcrepo-integration-ldp project. (enforce-victims-rule...enforce-victims-rule) http://git.io/QmZDvg
* github-ff leaves
<pivotal-bot____>Andrew Woods added comment: "I squashed this PR update into PR-545 to fix the vulnerability issue.
https://github.com/fcrepo4/fcrepo4/pull�" https://www.pivotaltracker.com/story/show/72875022
Andrew Woods delivered "Add enforce-victims-rule plugin (?)" https://www.pivotaltracker.com/story/show/7287502217:58
* github-ff joins17:59
[fcrepo4] awoods closed pull request #545: Add enforce-victims-rule to prevent shipping with vulnerable dependencies (master...enforce-victims-rule) http://git.io/RGW_WQ
* github-ff leaves
<pivotal-bot____>Andrew Woods added comment: "Resolved with: https://github.com/fcrepo4/fcrepo4/commit/2a18f12d0fd1b870366a8e744ecf3809d4db33cb" https://www.pivotaltracker.com/story/show/72875022
* github-ff joins
[fcrepo4] awoods closed pull request #558: Exclude httpclient from fcrepo-integration-ldp project. (enforce-victims-rule...enforce-victims-rule) http://git.io/QmZDvg
* github-ff leaves
<pivotal-bot____>Andrew Woods delivered "Remove fcr:sitemaps" https://www.pivotaltracker.com/story/show/8055416618:05
* github-ff joins
[fcrepo4] awoods closed pull request #544: Remove Sitemaps JAX-B resources (master...no-sitemaps) http://git.io/1020aA
* github-ff leaves
<awoods>cbeer: I will be back in about 2hours... and plan on trying to clear out the tickets that need review tonight and this weekend.18:06
* travis-ci joins18:07
fcrepo4/fcrepo4#2820 (html-binary-redirect - c1e0846 : Chris Beer): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/63fb6a9c2bb7...c1e0846e9867
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38310625
* travis-ci leaves
* travis-ci joins18:12
fcrepo4/fcrepo4#2822 (master - 2a18f12 : Chris Beer): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/91c464d067ef...2a18f12d0fd1
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38310781
* travis-ci leaves
* acoburn1 leaves18:13
* acoburn joins18:14
* travis-ci joins18:17
fcrepo4/fcrepo4#2824 (master - f5353ac : Andrew Woods): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/2a18f12d0fd1...f5353ac10ee3
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38311519
* travis-ci leaves
* acoburn leaves18:27
* acoburn joins18:28
* acoburn leaves18:33
<pivotal-bot____>Chris Beer deleted "Update node COPY/MOVE to be cross-workspace ready" https://www.pivotaltracker.com/story/show/6088732218:34
Chris Beer deleted "Define fcrepo4-specific Prefer header values for adding additional sources of triples " https://www.pivotaltracker.com/story/show/69468510
* nikhiltri leaves19:05
* scossu leaves
* MohamedAR leaves19:14
<pivotal-bot____>Chris Beer deleted "SimpleObserver only works for the default workspace" https://www.pivotaltracker.com/story/show/7229109219:25
Chris Beer deleted "Add ability to rename namespace prefixes after-the-fact" https://www.pivotaltracker.com/story/show/72087274
Chris Beer deleted "Consider reporting issue from https://github.com/futures/fcrepo4/pull/212 upstream" https://www.pivotaltracker.com/story/show/6248078019:28
Chris Beer added "Add (configurable) support for LDP requirement that LDP servers SHOULD require If-Match for PUT requests " https://www.pivotaltracker.com/story/show/8095076020:05
Chris Beer estimated "Add (configurable) support for LDP requirement that LDP servers SHOULD require If-Match for PUT requests " as 1 point https://www.pivotaltracker.com/story/show/80950760
Chris Beer started "Add (configurable) support for LDP requirement that LDP servers SHOULD require If-Match for PUT requests " https://www.pivotaltracker.com/story/show/80950760
Chris Beer finished "Add (configurable) support for LDP requirement that LDP servers SHOULD require If-Match for PUT requests " https://www.pivotaltracker.com/story/show/8095076020:27
* github-ff joins
[fcrepo4] cbeer created ldp-put-required (+1 new commit): http://git.io/GFjUhw
fcrepo4/ldp-put-required 3dd1eac Chris Beer: Add Spring-based configuration to toggle LDP strict handling of PUT requests...
* github-ff leaves
<pivotal-bot____>Chris Beer added comment: "https://github.com/fcrepo4/fcrepo4/pull/559" https://www.pivotaltracker.com/story/show/80950760
* github-ff joins20:28
[fcrepo4] cbeer opened pull request #559: Add Spring-based configuration to toggle LDP strict handling of PUT requests (master...ldp-put-required) http://git.io/ehmXpw
* github-ff leaves
* travis-ci joins20:38
fcrepo4/fcrepo4#2825 (ldp-put-required - 3dd1eac : Chris Beer): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/commit/3dd1eacf288f
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38319225
* travis-ci leaves
* jcoyne joins20:47
<pivotal-bot____>Andrew Woods delivered "Add (configurable) support for LDP requirement that LDP servers SHOULD require If-Match for PUT requests " https://www.pivotaltracker.com/story/show/8095076021:13
* github-ff joins
[fcrepo4] awoods closed pull request #559: Add Spring-based configuration to toggle LDP strict handling of PUT requests (master...ldp-put-required) http://git.io/ehmXpw
* github-ff leaves
<pivotal-bot____>Andrew Woods added "Datastreams in HTML view should have /fcr:metadata" https://www.pivotaltracker.com/story/show/8095240821:16
Andrew Woods edited "Datastreams in HTML view should have /fcr:metadata" https://www.pivotaltracker.com/story/show/80952408
* travis-ci joins21:24
fcrepo4/fcrepo4#2827 (master - b028e48 : Andrew Woods): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/f5353ac10ee3...b028e480b461
Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/38320871
* travis-ci leaves
* jcoyne leaves21:52
* jcoyne joins22:53
* awead leaves23:03
* awead joins23:04
* awead leaves23:08
* f4jenkins leaves23:24
* f4jenkins joins23:28
* jcoyne leaves00:22
* ksclarke1 joins00:58
* ksclarke1 leaves01:46

Generated by Sualtam