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

Using timezone: Eastern Standard Time
* github-ff joins10:46
[fcrepo-module-auth-rbacl] awoods pushed 1 new commit to master: http://git.io/0mb1EA
fcrepo-module-auth-rbacl/master 2b2f694 Andrew Woods: Update based on impact of fcrepo shift from checked to runtime exceptions...
* github-ff leaves
* github-ff joins10:47
[fcrepo-module-auth-xacml] awoods pushed 1 new commit to master: http://git.io/Tptuqw
fcrepo-module-auth-xacml/master ca1a0fd Andrew Woods: Update based on impact of fcrepo shift from checked to runtime exceptions...
* github-ff leaves
<mohamed>Restarted logger. Quit for same reason as the last time - ping timeout10:48
<pivotal-bot____>Andrew Woods added comment: "Resolved with both: ""
https://github.com/fcrepo4/fcrepo-module-auth-rbacl/commit/2b2f6949d773d8f67934b4242e3685�" https://www.pivotaltracker.com/story/show/78913212
Andrew Woods delivered "Absorb impact of fcrepo shift from checked to runtime exceptions" https://www.pivotaltracker.com/story/show/78913212
<f4jenkins>Yippee, build fixed!10:49
Project fcrepo-module-auth-rbacl build #110: FIXED in 3 min 25 sec: http://jenkins.fcrepo.org/job/fcrepo-module-auth-rbacl/110/
awoods: Update based on impact of fcrepo shift from checked to runtime exceptions
<awoods>mohamed: what is your restart procedure? If you are not around it would be good for me to know.10:50
* ksclarke leaves11:04
* ksclarke joins11:07
* ajs6f1 joins11:09
* ajs6f2 joins11:11
* ajs6f1 leaves
* ajs6f leaves
<mohamed>awoods: ./control stop; ./control start from sualtam dir.11:12
<awoods>thanks, mohamed11:24
<ajs6f2>cbeer: It's definitely closer. We'll eventually see, I think, some kind of UpdateStrategy<T> extends Function<RdfStream, RdfStream> where T varies across Model, RdfStream, TypeforSparqlUpdateOperation, etc, depending on the kind of mutation we're inducing.11:39
* ajs6f2 leaves11:49
* ajs6f joins11:52
<cbeer>ajs6f: i need some java help.. say i have a `final Class<? extends RdfStream> context`12:09
ajs6f: and i want to make a Collection<Class<? extends RdfStream>>
<ajs6f>cbeer: Yeah?
<cbeer>ajs6f: Collection.singleton(context) wants to make a Collection<? extends <Class<? extends RdfStream>>>
<ajs6f>cbeer: Is that a problem?12:10
<cbeer>is it?
<ajs6f>I don't think so. ? extends <Class<? extends RdfStream>> is clearly a subtype of Class<? extends RdfStream>, right?
What are you going to do with this collection?12:11
<cbeer>ajs6f: i need to replace the varargs `final Class<? extends RdfStream>... context` with a collection. turns out the way we use it upstream, a varargs is just annoying to work with12:12
<ajs6f>Okay, but final Class<? extends RdfStream>... context is an array, right? What about Arrays.asList(context)?12:13
Or you mean you want to change the type of the parameter 'context'?12:14
<cbeer>ajs6f: the other thing that's factoring into this is the varargs makes converting some of our unit tests a massive pain.
<ajs6f>So you want to get rid of the vararg parameter?12:15
<cbeer>yes.
<ajs6f>How about just replacing it with Collection<Class<? extends RdfStream>>?
Then your use of singleton() should be fine.
As well as letting people pass in multiple guys when that is what they want to do.12:16
<cbeer>ok. i guess i just didn't know Collection<? extends <Class<? extends RdfStream>>> and Collection<Class<? extends RdfStream>> were equivalent.12:19
<ajs6f>Well, not equivalent, the firs tis a subtype of the second, but that works fine here. It's no different that passing in, say, a NodeRdfContext as a parameter in a method that wants an RdfStream, because NodeRdfContext is a subtype of RdfStream.
that's exactly the meaning of the first "extends" in ? extends <Class<? extends RdfStream>>12:20
As long as you only call methods on that passed-in thing that would be suitable for a Collection<Class<? extends RdfStream>>, your fine, and the comipler will tell you if you try to do something you shouldnt.12:21
<cbeer>ok. then i don't understand ? extends, but i don't think that's going to hurt me today.12:23
<ajs6f>I think you're fine, but i you've curious, check out:12:25
http://www.angelikalanger.com/GenericsFAQ/FAQSections/TypeArguments.html#Topic2
It does a good job explaining wilcards (the ?) and upper bounds (the extends X part).
If it helps, you can consider that you _could_ define the method like this:12:27
<T extends RdfStream> ReturnType myMethod (final Collection<Class<T>> myParams))12:28
and you wouldn't be using a wildcard there.12:29
You might use the wildcard because you don't really care what T is. You don't expect to return anything of type T.
You only care that it is a subtype of RdfStream.
In the more complicated signature, you don't care what the first ? represents. You don't intend to return any value of that type12:30
you only care that it is a subtype of Class<? extends RdfStream>.
Does that help?
<cbeer>we'll find out, i guess.12:39
<ajs6f>FULL STEAM AHEAD!12:40
<cbeer>ajs6f: ok, so that was the easy part of getTriples.. now, there's this method:12:42
https://github.com/fcrepo4/fcrepo4/blob/master/fcrepo-kernel-impl/src/main/java/org/fcrepo/kernel/impl/FedoraResourceImpl.java#L304
it's doing some pagination for us
i think we can kill it (now that the LDP sense of paging has shifted under us)
and try to reimplement it on top of our iterators
<ajs6f>+1. I'm not sure paging really belongs that far down in the kernel, anyway.12:43
<cbeer>good
<ajs6f>Kill 'em all, let James Gosling
sort 'em out.
* mickeroo_ leaves13:12
* mickeroo joins
* dwilcox joins13:26
* kaarefc joins13:35
<pivotal-bot____>Andrew Woods edited "Minor cleanup for storage policy types." https://www.pivotaltracker.com/story/show/7882111813:42
Andrew Woods accepted "Remove unused methods from core model classes" https://www.pivotaltracker.com/story/show/78815814
Andrew Woods accepted "Minor cleanup for storage policy types." https://www.pivotaltracker.com/story/show/78821118
Andrew Woods accepted "Expose RepositoryRuntimeException instead of the RepositoryException" https://www.pivotaltracker.com/story/show/78860776
Andrew Woods accepted "Update ISPN config files based on ModeShape 4.0-Beta" https://www.pivotaltracker.com/story/show/78475724
Andrew Woods accepted "Absorb impact of fcrepo shift from checked to runtime exceptions" https://www.pivotaltracker.com/story/show/78913212
* fasseg joins13:43
* dwilcox leaves13:57
* dwilcox joins14:03
* dwilcox leaves
* kaarefc leaves14:08
* dwilcox joins14:11
<awoods>fasseg: ping14:14
* github-ff joins14:32
[fcrepo4] cbeer created getTriples (+8 new commits): http://git.io/6XKP6g
fcrepo4/getTriples 08ce06f Christopher Beer: Rename getTriples to getPropertiesTriples
fcrepo4/getTriples 109cd42 Christopher Beer: Extend signature of replaceProperties to include the RdfStream of the object(s) being replaced
fcrepo4/getTriples 829e4f4 Christopher Beer: Add getTriples
* github-ff leaves
* github-ff joins14:33
[fcrepo4] cbeer opened pull request #475: Create getTriples member for converting a resource to RDF (master...getTriples) http://git.io/PCau8g
* github-ff leaves
* github-ff joins14:34
[fcrepo4] cbeer force-pushed getTriples from 0bf3eae to 29f60b7: http://git.io/F3nCpg
fcrepo4/getTriples 25f4787 Christopher Beer: Add explicit mix:versionable check in FedoraVersions instead of relying on an...
fcrepo4/getTriples d92efbe Christopher Beer: remove getPropertiesTriples
fcrepo4/getTriples c8a3a60 Christopher Beer: Remove getVersionTriples
* github-ff leaves
<cbeer>+495 / -1422.. so sad.14:35
* travis-ci joins14:43
[travis-ci] fcrepo4/fcrepo4#2361 (getTriples - 0bf3eae : Christopher Beer): The build passed.
[travis-ci] Change view : https://github.com/fcrepo4/fcrepo4/compare/08ce06f1fff1^...0bf3eae3841c
[travis-ci] Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/35462721
* travis-ci leaves
* travis-ci joins14:45
[travis-ci] fcrepo4/fcrepo4#2363 (getTriples - 29f60b7 : Christopher Beer): The build passed.
[travis-ci] Change view : https://github.com/fcrepo4/fcrepo4/compare/0bf3eae3841c...29f60b7d2539
[travis-ci] Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/35462851
* travis-ci leaves
* ajs6f leaves14:47
<f4jenkins>Project fcrepo4-client build #77: UNSTABLE in 2 min 39 sec: http://jenkins.fcrepo.org/job/fcrepo4-client/77/14:51
Project fcrepo-module-auth-rbacl build #111: UNSTABLE in 4 min 18 sec: http://jenkins.fcrepo.org/job/fcrepo-module-auth-rbacl/111/14:53
Project fcrepo4 build #2138: UNSTABLE in 11 min: http://jenkins.fcrepo.org/job/fcrepo4/2138/15:00
* gregjansen1 joins15:03
* gregjansen1 leaves
* github-ff joins15:05
[fcrepo4] mikedurbin opened pull request #476: Shutdown hook (master...shutdown-hook) http://git.io/MQ89fw
* github-ff leaves
<pivotal-bot____>Mike Durbin added comment: "I rebased and reintroduced the unit test. I look over the differences between the JcrRepositoryFactory and th�" https://www.pivotaltracker.com/story/show/6645409615:06
Mike Durbin finished "Ensure a clean shutdown of the repository on container shutdown." https://www.pivotaltracker.com/story/show/66454096
Mike Durbin started "Automate running sqarql-recipes" https://www.pivotaltracker.com/story/show/7252414015:07
* awead leaves15:08
* fasseg leaves15:32
* awead joins15:36
* whikloj leaves16:01
<pivotal-bot____>Stefano Cossu added "Ad TOC to long RESTful API reference pages" https://www.pivotaltracker.com/story/show/7895353216:20
* nikhiltri joins16:29
<pivotal-bot____>Stefano Cossu added "MOVE HTTP method throws authentication error" https://www.pivotaltracker.com/story/show/7895549416:41
Stefano Cossu edited "MOVE HTTP method throws authentication error" https://www.pivotaltracker.com/story/show/78955494
* gregjansen joins16:44
* mikeAtUVa leaves16:55
* nikhiltri leaves17:00
<pivotal-bot____>Andrew Woods delivered "Ad TOC to long RESTful API reference pages" https://www.pivotaltracker.com/story/show/7895353217:08
Stefano Cossu added comment: "Awesome, @awoods - and in record time. Thanks." https://www.pivotaltracker.com/story/show/7895353217:11
Stefano Cossu accepted "Ad TOC to long RESTful API reference pages" https://www.pivotaltracker.com/story/show/7895353217:13
* gregjansen leaves17:17
* nikhiltri joins17:30
* mohamed leaves17:31
<pivotal-bot____>Chris Beer started "getChildren() on FedoraResource" https://www.pivotaltracker.com/story/show/7161129817:35
Chris Beer edited "getChildren() on FedoraResource" https://www.pivotaltracker.com/story/show/7161129817:36
Chris Beer unstarted "getChildren() on FedoraResource" https://www.pivotaltracker.com/story/show/71611298
Chris Beer edited "getChildren() on FedoraResource" https://www.pivotaltracker.com/story/show/71611298
Chris Beer added "Refactor FedoraResource#get*Triples into a single getTriples method" https://www.pivotaltracker.com/story/show/78960386
Chris Beer finished "Refactor FedoraResource#get*Triples into a single getTriples method" https://www.pivotaltracker.com/story/show/78960386
Chris Beer added comment: "https://github.com/fcrepo4/fcrepo4/pull/475" https://www.pivotaltracker.com/story/show/7896038617:37
* github-ff joins18:03
[fcrepo4] cbeer created resource-interfaces (+1 new commit): http://git.io/-N-eRw
fcrepo4/resource-interfaces 3f677f8 Christopher Beer: Extract JCR-agnostic resource interfaces out of FedoraResource/Datastream
* github-ff leaves
* github-ff joins
[fcrepo4] cbeer opened pull request #477: Extract JCR-agnostic resource interfaces out of FedoraResource/Datastream (getTriples...resource-interfaces) http://git.io/476T1w
* github-ff leaves
* travis-ci joins18:07
[travis-ci] fcrepo4/fcrepo4#2366 (resource-interfaces - 3f677f8 : Christopher Beer): The build has errored.
[travis-ci] Change view : https://github.com/fcrepo4/fcrepo4/commit/3f677f83a42f
[travis-ci] Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/35481195
* travis-ci leaves
* awead leaves18:13
* awead joins18:15
* nikhiltri leaves18:18
<f4jenkins>Yippee, build fixed!18:19
Project fcrepo4 build #2139: FIXED in 16 min: http://jenkins.fcrepo.org/job/fcrepo4/2139/
Yippee, build fixed!18:22
Project fcrepo4-client build #78: FIXED in 2 min 20 sec: http://jenkins.fcrepo.org/job/fcrepo4-client/78/
* ksclarke leaves
<f4jenkins>Yippee, build fixed!18:24
Project fcrepo-module-auth-rbacl build #112: FIXED in 4 min 10 sec: http://jenkins.fcrepo.org/job/fcrepo-module-auth-rbacl/112/
* awead leaves18:27
<f4jenkins>Project fcrepo-message-consumer build #656: ABORTED in 3 hr 50 min: http://jenkins.fcrepo.org/job/fcrepo-message-consumer/656/18:39
* scossu1 leaves18:41
* dwilcox leaves18:42
<f4jenkins>Project fcrepo-message-consumer build #657: SUCCESS in 5 min 14 sec: http://jenkins.fcrepo.org/job/fcrepo-message-consumer/657/18:44
<pivotal-bot____>Andrew Woods unstarted "Add OPTIONS response for FedoraContent" https://www.pivotaltracker.com/story/show/7812876818:54
Andrew Woods edited "Add OPTIONS response for FedoraContent" https://www.pivotaltracker.com/story/show/78128768
* nikhiltri joins
* nikhiltri leaves18:55
* nikhiltri joins18:56
* nikhiltri leaves18:57
* nikhiltri joins18:58
* nikhiltri leaves19:05
* awead joins20:04
* awead leaves20:59
* github-ff joins21:02
[fcrepo4] cbeer force-pushed resource-interfaces from 3f677f8 to 8f746de: http://git.io/Bv4McA
fcrepo4/resource-interfaces 8f746de Christopher Beer: Extract JCR-agnostic resource interfaces out of FedoraResource/Datastream
* github-ff leaves
* awead joins21:04
* travis-ci joins21:11
[travis-ci] fcrepo4/fcrepo4#2368 (resource-interfaces - 8f746de : Christopher Beer): The build passed.
[travis-ci] Change view : https://github.com/fcrepo4/fcrepo4/compare/3f677f83a42f...8f746de4fac5
[travis-ci] Build details : http://travis-ci.org/fcrepo4/fcrepo4/builds/35493402
* travis-ci leaves
* awead leaves21:18
* ksclarke joins21:25
* mohamed joins21:47
* ruebot leaves21:48
* ruebot joins21:54
* mohamed leaves22:02
* fcrepo-bot joins22:59

Generated by Sualtam