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

Using timezone: Eastern Standard Time
* dhlamb joins07:46
* acoburn joins08:08
* ruebot leaves09:02
* ksclarke joins09:13
* ajs6f joins09:39
* ruebot joins09:48
* ruebot leaves
* ruebot joins
* ksclarke leaves09:51
* ksclarke joins10:07
* yinlin joins10:12
* whikloj joins10:14
* acoburn leaves10:47
<f4jenkins>Yippee, build fixed!10:52
Project fcrepo4-T2 build #172: FIXED in 4 min 58 sec: http://jenkins.fcrepo.org/job/fcrepo4-T2/172/
<Nianli> dwilcox: yesterday: working on FCREPO-1386, today: continue working on this issue, hoping I can finish it today. Blocker: passing command line args to bean, help me if you knows!10:58
* acoburn joins
* escowles joins
* dshalvi joins10:59
<escowles>ruebot: dhlamb: coming to the call?11:02
https://wiki.duraspace.org/display/FF/2015-03-12+-+Fedora+Tech+Meeting
<ajs6f>We want Islandora!
<whikloj>dwilcox: Yesterday completed PR for FCREPO-1363, also working on FCREPO-1361. Today continue working on FCREPO-1361.11:05
<ruebot>escowles: so. in san diego at pasig. did dhlamb forget to call in?11:38
<whikloj>ajs6f: ok so it appears that to properly through an error before executing any updates, I will need to reverse the logic of the JcrPropertyListener and test each statement first.
<ruebot>s/so/sorry/11:39
<escowles>ruebot: yep, we just skipped your standing agenda item and had a nice, friendly discussion of non-container LDPRs
<ajs6f>whikloj: I don't have that code in front of me, but it seems reasonable that yes, you would need to test each triple and check to see that its subject is IdentifierConverter.inDomain().
<ruebot>escowles: please tell me you just read the lyrics to epic by faith no more?11:40
<ajs6f>whikloj: Although I don't see why you wouldn't do that right in JcrPropertyListener.
<escowles>ruebot++
<whikloj>ajs6f: https://github.com/fcrepo4/fcrepo4/blob/master/fcrepo-kernel-impl/src/main/java/org/fcrepo/kernel/impl/utils/JcrPropertyStatementListener.java11:41
<ajs6f>whikloj: Seems like you could just change:
if (!idTranslator.inDomain(subject) && !subject.isAnon()) {11:42
return;
}
to throw an appropriate exception.
<whikloj>ajs6f: right but if that doesn't happen until the second statement then we will have partially completed the execution. Instead of not executing anything.11:43
<ajs6f>whikloj: you haven't done anything until you do session.save();
whikloj: you're just building up changes that get written then.11:44
whikloj: and to be clear, session.save() is executed at the HTTP layer, way, way above this stuff.11:45
<whikloj>ajs6f: ok, I get it
<ajs6f>whikloj: Rock on.
<whikloj>ajs6f: thanks a ton that was a huge help.11:46
<ajs6f>whikloj: Is there anything I about which I can tell you "No."? I'm really good at that.
<whikloj>ajs6f: can I have a million dollars?
<ajs6f>whikloj: -111:47
Veto.
<whikloj>doh
ajs6f: You want to give me an accurate error message to return.11:49
ajs6f: When I was playing with loading them I threw "Asked to translate a resource http://example.org/foo that doesn't match the URI template"
But this is more like "Subject <foo> is not in the domain of this repository."11:50
<ajs6f>whikloj: Good question. How about "Update included RDF subjects that were not in the domain of this repository."
<whikloj>sounds good
<ajs6f>whikloj: Great minds think alike.
<barmintor>I think I see the root of a lot of my discomfort lately: Fedora 4 doesn’t makes sense to me, I guess any more.11:56
<ajs6f>barmintor: Weird. That's exactly what I was thinking yestereve.11:57
barmintor: I mean, it makes sense, just not as the set of ideas I knew as Fedora.
* acoburn leaves12:00
<barmintor>escowles, whikloj is there a reason not to split repository.rdf up into smaller pieces?12:03
<escowles>barmintor: no reason in principle -- are you thinking of splitting it up along functional lines?12:04
<whikloj>+1 escowles
<barmintor>escowles: yes, I want to be able to argue with ajs6f about the actual Fedora model, and it’s hard to dig it out of this document
<ajs6f>barmintor/escowles: I was under the understanding that we definitely are going to split up repository.rdf, as the larger project of partiitioning the API goes forward. (at a glacial pace).
minutes up: https://wiki.duraspace.org/display/FF/2015-03-12+-+Fedora+Tech+Meeting12:11
* ajs6f leaves12:23
* dhlamb_ joins12:26
* ajs6f joins12:27
tecoripa++12:29
* dhlamb leaves
* bljenkins leaves12:36
* github-ff joins12:43
[fcrepo4] whikloj opened pull request #747: Fail on Sparql update for subjects outside repository domain (master...FCREPO-1361) http://git.io/pXbv
* github-ff leaves
<ajs6f>afk bbl13:09
* ajs6f leaves
* escowles leaves13:24
* dhlamb joins13:34
* dhlamb_ leaves13:35
* acoburn joins13:39
* acoburn leaves14:01
* github-ff joins
[fcrepo4-release-tests] yinlinchen opened pull request #8: add activemq dynamic port (master...fcrepo-1366) http://git.io/p1wT
* github-ff leaves
* acoburn joins14:02
* yinlin leaves14:17
* pgwillia joins14:32
* dshalvi leaves14:43
* dhlamb leaves15:38
* acoburn1 joins15:50
* acoburn leaves15:51
* acoburn1 leaves15:53
* acoburn joins
* ajs6f joins16:06
* acoburn leaves16:33
* whikloj leaves17:54
* ksclarke leaves18:10
<ajs6f>out for the day18:17
* ajs6f leaves
* pgwillia leaves18:20
* dwilcox joins18:52
* jgpawletko leaves20:36
* jgpawletko joins20:42
* dwilcox leaves21:18
* ksclarke joins21:38
* dhlamb joins21:56
* ksclarke leaves22:29
* ksclarke joins22:44
* jgpawletko leaves23:14
* dhlamb leaves00:10

Generated by Sualtam