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

Using timezone: Eastern Standard Time
* github-ff joins00:49
[fcrepo-jms-indexer-pluggable] ksclarke opened pull request #39: updated checkstyle dep and moved config into pluginManagement (master...update-checkstyle-dep-70648050) http://git.io/uJ8Ctg
* github-ff leaves
<pivotal-bot>Kevin Clarke added comment: "Updated checkstyle dependency and moved its configuration into pluginManagement00:50
https://github.com/futures..." https://www.pivotaltracker.com/story/show/70648050
Kevin Clarke started "Update checkstyles for fcrepo-jms-indexer-pluggable" https://www.pivotaltracker.com/story/show/70648050
Kevin Clarke finished "Update checkstyles for fcrepo-jms-indexer-pluggable" https://www.pivotaltracker.com/story/show/70648050
Kevin Clarke estimated "Refactor fedoraUser AuthZ expectation" as 3 points https://www.pivotaltracker.com/story/show/7030626800:52
Kevin Clarke estimated "How many objects can F4 hold?" as 3 points https://www.pivotaltracker.com/story/show/7064865800:53
Kevin Clarke estimated "Tighten code styling" as 5 points https://www.pivotaltracker.com/story/show/70648784
* ksclarke leaves01:10
* dwilcox joins07:58
<pivotal-bot>Kai Sternad finished "Write an integration test to ensure that transactions cannot be hijacked." https://www.pivotaltracker.com/story/show/6601468608:14
* mikeAtUVa joins08:41
* ksclarke joins09:03
<pivotal-bot>Andrew Woods estimated "Move git repos to new git organization" as 2 points https://www.pivotaltracker.com/story/show/7067134209:13
* github-ff joins09:18
[fcrepo4] mikedurbin opened pull request #341: Added session.logout() where missing from REST call implementations. (master...lock-view-bug) http://git.io/ZUG6fw
* github-ff leaves
<pivotal-bot>Mike Durbin added comment: "PR https://github.com/futures/fcrepo4/pull/341" https://www.pivotaltracker.com/story/show/7059766609:19
Mike Durbin finished "Getting lock metadata multiple times in a session throws exception." https://www.pivotaltracker.com/story/show/70597666
Mike Durbin started "Add mix:versionable mixin (if missing) when user attempts to snapshot a version of a node." https://www.pivotaltracker.com/story/show/70289498
Andrew Woods added comment: "new ticket: #70648658" https://www.pivotaltracker.com/story/show/6945415209:20
<awoods>ksclarke: in your last red graph: https://wiki.duraspace.org/display/FF/Flat+Hierarchies+Testing09:24
ksclarke: where there datastreams? or just naked objects?
ksclarke: were there datastreams? or just naked objects?
<ksclarke>datastreams, 1MB... I'll update to indicate
<pivotal-bot>Andrew Woods delivered "ExecutionError while trying to ingest 500k records" https://www.pivotaltracker.com/story/show/6945415209:27
Andrew Woods accepted "ExecutionError while trying to ingest 500k records" https://www.pivotaltracker.com/story/show/6945415209:35
* cjcolvar joins09:52
* dwilcox leaves09:59
* cjcolvar1 joins10:02
* dwilcox joins10:04
<pivotal-bot>Andrew Woods delivered "Update upstream dependencies" https://www.pivotaltracker.com/story/show/70676928
* github-ff joins
[fcrepo4] awoods deleted dep-update at 8bed92c: http://git.io/m-GglA
* github-ff leaves
* cjcolvar leaves
* cjcolvar1 leaves10:14
* longshou joins
* tecoripa joins10:17
<pivotal-bot>Longshou Situ started "Clean-up Fedora wiki main page" https://www.pivotaltracker.com/story/show/7067361210:21
* travis-ci joins
[travis-ci] futures/fcrepo4#1896 (master - e592293 : Andrew Woods): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/d82a27ff9df3...e592293cf549
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24537512
* travis-ci leaves
<pivotal-bot>Scott Prater started "Address "Critical" Sonar Issues" https://www.pivotaltracker.com/story/show/7058481810:22
Kevin Clarke started "Turn on checkstyle checking for fcrepo4's test classes" https://www.pivotaltracker.com/story/show/6935519410:26
* cjcolvar joins10:27
<awoods>longshou: when you have a moment, could you please estimate the pivotal tickets to which your name is assigned?10:28
* github-ff joins10:32
[fcrepo4] escowles created reduced-logging (+2 new commits): http://git.io/0ud6FA
fcrepo4/reduced-logging 9783d73 Esmé Cowles: Configuring logging to INFO level by default, reducing error messages and stacktraces, changing handled exception logging to info instead of warn
fcrepo4/reduced-logging 5d9656c Esmé Cowles: Standardizing logging levels and making them configurable with system properties
* github-ff leaves
<longshou>awoods: It’s hard for me to estimate them at the moment since I haven’t touched the codebase for those two tickets yet. Please go ahead and estimate them for me for now. Thanks.10:33
<awoods>longshou: maybe ksclarke can help out.10:34
* github-ff joins
[fcrepo4] escowles opened pull request #342: Reduced logging (master...reduced-logging) http://git.io/H5fu_w
* github-ff leaves
<pivotal-bot>Esme Cowles added comment: "https://github.com/futures/fcrepo4/pull/342" https://www.pivotaltracker.com/story/show/6985953010:35
Esme Cowles finished "Reduce build and runtime logging verbosity" https://www.pivotaltracker.com/story/show/69859530
<ksclarke>longshou: I find it difficult to estimate as well... I just make a best guess (trying to err towards being more conservative since I know I'm an optimistic guesser)
awoods: just interesting tidbit... this is one of fogbugz strengths... keeps track of whether a person has been an optimistic or pessimistic guesser and adjusts project estimates with that history in mind10:36
of course that also requires accurate start/finish actions (so the real time can be measured against the estimated time)10:37
<awoods>ksclarke: that would be a nice feature.
ksclarke: I guess we could do that manually
<ksclarke>longshu: but I just work from the assumption it's going to take me longer to do everything because of my unfamiliarity with most of the project10:38
longshu: I actually considered re-estimating some of the task that has been already estimated just to add more time to them (but didn't)
awoods: do that manually like the start/finish (which I think we do) or do the comparison of actual time vs. estimated (the latter sounds like a lot of work)10:39
sort of for start/finish... I'd forgotten to start the last ticket until I saw someone else start one in here10:40
<awoods>ksclarke: I was thinking the latter, which would be work.
<longshou>Kevin: That’s Okay. We often under estimated the tickets. Thanks a lot.
<ksclarke>awoods: yeah, though doing that manually sounds like a lot of work to me10:41
<pivotal-bot>Longshou Situ added comment: "@" https://www.pivotaltracker.com/story/show/70648666
* gregjansen joins
<ksclarke>awoods: plus having it as a part of the system sort of abstracted away the (possibly perceived) judgement10:43
anyway :-)
<benpennell>mikeAtUVa: "Creating a version of a node that has a large number of descendants" does that mean lots of datastreams or lots of children objects?10:49
<mikeAtUVa>benpennell: Perhaps both.... plus perhaps a small number of children (one container node) but a large number of descendants.10:50
benpennel: one thing I'm getting at is that if you use the auto-naming for child nodes or elements, versioning behaves differently than without it right now. We don't know how costly that is.
benpennel: the things we haven't really tested are the combinations of: a versionable node with lots of versionable descendants... a versionable node with lots of descendants under non-versionable nodes... and whether data size vs. node count makes affects the performance.10:53
* travis-ci joins10:54
[travis-ci] futures/fcrepo4#1898 (reduced-logging - 5d9656c : Esmé Cowles): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/9783d736c428^...5d9656cd1c13
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24540244
* travis-ci leaves
<benpennell>mikeAtUVa: okay, and is it the creation of a new version of the node that we are primarily interesting in measuring in these conditions? both in terms of execution time and disk usage10:55
* github-ff joins10:57
[fcrepo4] kaisternad opened pull request #343: Fcrepo 66014686 (master...fcrepo-66014686) http://git.io/1NrYBQ
* github-ff leaves
<mikeAtUVa>benpennell: I think so, though perhaps people are also concerned with the potential impact of regular operations. (do updates or node creation take longer when a node has the mix:versionable mixin?)
benpennell: if at the end of the testing we would know a) how much overhead version creation is, relative to regular updates, b) whether it scales linearly or otherwise c) whether certain arrangments of versionable and non-versionable nodes are problematic (ie, scale worse) d) whether just making a node versionable slows down other operations...11:00
<benpennell>okay thanks mike, I'm going to add that to the ticket11:01
<mikeAtUVa>benpennel: I know that's asking a lot, and perhaps there's even more to wonder about. Hopefully if there are conditions that aren't scalable, (ie, unversioned nodes below versioned nodes, which I think are problematic) it'll guide our updates to default behavior.
<pivotal-bot>Benjamin Pennell added comment: "mikeAtUVa11:06
one thing I'm getting at is that if you use the auto-naming for child nodes or elements, vers..." https://www.pivotaltracker.com/story/show/70288940
Esme Cowles added comment: "The current build (b89273c5edfa340760495ebdb0d0dcd51044cece) succeeds for me." https://www.pivotaltracker.com/story/show/7067342611:11
Scott Prater added comment: "Same build succeeds for me, too." https://www.pivotaltracker.com/story/show/7067342611:13
Gregory Jansen added comment: "This old repository will be deleted shortly, after anything useful has been copied to the new XACML proje..." https://www.pivotaltracker.com/story/show/6239960811:16
Esme Cowles edited "Federation: How many files can be managed?" https://www.pivotaltracker.com/story/show/7072457211:23
Benjamin Pennell edited "Create XACML AuthZ Project" https://www.pivotaltracker.com/story/show/7068813411:24
Andrew Woods edited "Create CND to express XACML policy structure" https://www.pivotaltracker.com/story/show/7068952011:25
Esme Cowles estimated "Federation: How many files can be managed?" as 3 points https://www.pivotaltracker.com/story/show/70724572
Andrew Woods edited "Create a XACML AuthZ delegate and finder module stubs" https://www.pivotaltracker.com/story/show/7068886811:26
Scott Prater edited "Create XACML policies that enforce basic roles of reader, writer and admin" https://www.pivotaltracker.com/story/show/7068896211:27
Mike Durbin edited "Create XACML policies for non-role-based use cases" https://www.pivotaltracker.com/story/show/7068937411:29
Gregory Jansen estimated "XACML SubjectAttributeFinderModule" as 5 points https://www.pivotaltracker.com/story/show/7069591611:38
Gregory Jansen estimated "XACML resource finder module" as 3 points https://www.pivotaltracker.com/story/show/70690002
Gregory Jansen estimated "XACML Resource Predicate Attribute Finder Module" as 3 points https://www.pivotaltracker.com/story/show/70690150
Gregory Jansen estimated "Implement XACML policy finder module" as 3 points https://www.pivotaltracker.com/story/show/70689680
Gregory Jansen estimated "Create XACML policies for non-role-based use cases" as 3 points https://www.pivotaltracker.com/story/show/70689374
Gregory Jansen edited "Create XACML policies for non-role-based use cases" https://www.pivotaltracker.com/story/show/7068937411:39
Gregory Jansen estimated "XACML EnvironmentAttributeFinderModule" as 2 points https://www.pivotaltracker.com/story/show/70696018
<cbeer>escowles: ping?11:40
<pivotal-bot>Kevin Clarke edited "Refactor kernel/kernel-api names and packages" https://www.pivotaltracker.com/story/show/70718582
<escowles>cbeer: yep?
<cbeer>escowles: i think you had some idea about how to deal with the referential integrity errors we were getting in AF deletes at the fcrepo4 level?11:41
<pivotal-bot>Gregory Jansen added "Implement integration test for basic roles XACML policies" https://www.pivotaltracker.com/story/show/7077180611:42
Gregory Jansen edited "Implement integration test for basic roles XACML policies" https://www.pivotaltracker.com/story/show/70771806
<escowles>i was thinking we would either a) manually delete them, or b) use transactions -- but i think there was another option someone else had of making them weakreferences
<cbeer>escowles: so on DELETE /abc, we go clean up the inbound references too?
<escowles>cbeer: right -- basically implementing cascading delete11:43
<cbeer>escowles: works for me. i figured out how to get the inbound references yesterday, so that shouldn't be too hard
<escowles>i think jcoyne also thought we could convert the refs to URIs instead (break the link, but keep the info)
<pivotal-bot>Gregory Jansen added "Integration tests for non-role example XACML policies" https://www.pivotaltracker.com/story/show/7077201011:44
Gregory Jansen estimated "Integration tests for non-role example XACML policies" as 3 points https://www.pivotaltracker.com/story/show/70772010
<escowles>i think cascading delete is fine -- esp. if we can get the inbound links pretty easily
<cbeer>escowles: what he didn't know before, though, is if you have real references, you can https://github.com/futures/fcrepo4/pull/340
<pivotal-bot>Gregory Jansen edited "Integration tests for non-role example XACML policies" https://www.pivotaltracker.com/story/show/70772010
<cbeer>and cut out a query to solr
<pivotal-bot>Gregory Jansen edited "Integration tests for basic role XACML policies" https://www.pivotaltracker.com/story/show/70771806
<escowles>cbeer: right -- one wrinkle was that we were using solr to find the links, so transactions got screwy because fedora and solr would get out of sync11:45
so if we don't need to use solr, then this starts to look more reasonable
<pivotal-bot>Mike Durbin edited "Audit and enforce JCR leakages." https://www.pivotaltracker.com/story/show/6616306611:47
Esme Cowles edited "Revisit Holiday-Release tests" https://www.pivotaltracker.com/story/show/7072460411:50
Longshou Situ edited "How many concurrent users?" https://www.pivotaltracker.com/story/show/7072462211:51
Chris Beer added "NodeService#deleteObject should also clean up inbound references to the object" https://www.pivotaltracker.com/story/show/7077294211:52
Chris Beer started "NodeService#deleteObject should also clean up inbound references to the object" https://www.pivotaltracker.com/story/show/70772942
Scott Prater edited "How many concurrent users?" https://www.pivotaltracker.com/story/show/7072462211:57
Scott Prater edited "Load test CRUD operations with concurrent users" https://www.pivotaltracker.com/story/show/70724622
<tecoripa>awoods: for some of these Sonar exceptions, I'm seeing a bit of exception handling used for flow control11:58
* scossu joins
<tecoripa>awoods: some I can address, or talk over with the developer
there's one (the first one) that baffles me a little, though:11:59
<awoods>tecoripa: let's see
<tecoripa>just a sec... pulling up git
https://github.com/futures/fcrepo4/blob/master/fcrepo-kernel/src/main/java/org/fcrepo/kernel/TxAwareSession.java#L7512:00
and there's actually a test that checks for this12:01
TxAwareSessionTest.shouldThrowUnderlyingException
<awoods>tecoripa: what does sonar want?
<tecoripa>well, sonar wants the exact exception to be thrown, or logged12:02
so I refactored it to take out the try/catch, and have the method throw InvocationTargetException, IllegalAccessException
<awoods>tecoripa: or logging (DEBUG) what is going on makes sense.12:03
* nikhiltri joins
<tecoripa>that's an option... though it breaks the practice of logging or throwing an exception, but not doing both.
In this case, since we're just throwing the cause, I suppose we can fudge it
there's just the larger issue of using exceptions for flow control...12:04
here's the test: https://github.com/futures/fcrepo4/blob/master/fcrepo-kernel/src/test/java/org/fcrepo/kernel/TxAwareSessionTest.java#L8312:05
<awoods>tecoripa: The way I read the logic, we are throwing the specific exception if one is raised.
tecoripa: is that "using exceptions for flow control"?12:06
<tecoripa>we're throwing the root cause of the exception, not the exception itself... and that is tested later, to determine if a path is not found or not, or so it seems
* longshou leaves
<tecoripa>I suppose that's not really flow control, in this case.12:07
jus playing a bit fast and loose with exceptuion handling
I'll put in a log.DEBUG to keep sonar quiet, and move on
<awoods>tecoripa: I'm ok with that.12:08
<tecoripa>cool. thanks
I took a brief look at some of the other exceptions (all the rest of the Sonar warnings are exception-handling ones), I'll probably come back with more questions12:09
<awoods>tecoripa: several looked like swallowed exceptions, that could be debugged()
<tecoripa>yeah... some are mikeAtUVa's I'll ping him about them12:10
<awoods>tecoripa: mikeAtUVa is a sly one.12:11
<pivotal-bot>Andrew Woods added "Bump dependency versions in fcrepo-indexer-jms-pluggable" https://www.pivotaltracker.com/story/show/70774640
Andrew Woods started "Bump dependency versions in fcrepo-indexer-jms-pluggable" https://www.pivotaltracker.com/story/show/70774640
<tecoripa>awoods: I wonder what he does with all those swallowed exceptions. Sell them on EBay to penurious Java programmers?12:12
* gregjansen leaves12:13
<ksclarke>grr, seems my eclipse renaming of tests in kernel and kernel-api a month ago did not maintain git history12:30
* ksclarke feels a little uncomfortable adding @author tags for his name to these tests12:31
<awoods>ksclarke: join the club12:37
* nikhiltri leaves
<ksclarke>:-)12:38
* nikhiltri joins
<cbeer>simple solution: just use git history to tell who's edited a file?12:43
heresy, i know.
* github-ff joins
[fcrepo-jms-indexer-pluggable] awoods pushed 1 new commit to master: http://git.io/6AL5UA
fcrepo-jms-indexer-pluggable/master 8ed3807 Andrew Woods: Bump project dependencies...
* github-ff leaves
<awoods>cbeer: It would be nice if git history magically showed up in javadocs.12:44
<pivotal-bot>Andrew Woods added comment: "Resolved with: https://github.com/futures/fcrepo-jms-indexer-pluggable/commit/8ed3807fd4022ad2910c265b51371..." https://www.pivotaltracker.com/story/show/7077464012:45
Andrew Woods delivered "Bump dependency versions in fcrepo-indexer-jms-pluggable" https://www.pivotaltracker.com/story/show/70774640
<cbeer>can't say i've ever thought to use javadocs to figure out who wrote something.
<awoods>cbeer: others? I certainly appreciate seeing who has written something from the javadocs... not so much that I have been investigating who the author is.12:46
* github-ff joins12:54
[fcrepo-jms-indexer-pluggable] awoods pushed 1 new commit to master: http://git.io/0q2bnA
fcrepo-jms-indexer-pluggable/master d7c3197 Kevin S. Clarke: updated checkstyle dep and moved config into pluginManagement
* github-ff leaves
* github-ff joins12:55
[fcrepo-jms-indexer-pluggable] awoods closed pull request #39: updated checkstyle dep and moved config into pluginManagement (master...update-checkstyle-dep-70648050) http://git.io/uJ8Ctg
* github-ff leaves
<pivotal-bot>Andrew Woods added comment: "Resolved with: https://github.com/futures/fcrepo-jms-indexer-pluggable/commit/d7c3197c14db97ff1ed6246ee1124..." https://www.pivotaltracker.com/story/show/70648050
Andrew Woods delivered "Update checkstyles for fcrepo-jms-indexer-pluggable" https://www.pivotaltracker.com/story/show/70648050
* edInCo joins
* travis-ci joins12:57
[travis-ci] futures/fcrepo-jms-indexer-pluggable#111 (master - 8ed3807 : Andrew Woods): The build has errored.
[travis-ci] Change view : https://github.com/futures/fcrepo-jms-indexer-pluggable/compare/b89273c5edfa...8ed3807fd402
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo-jms-indexer-pluggable/builds/24552382
* travis-ci leaves
* longshou joins
* travis-ci joins13:09
[travis-ci] futures/fcrepo-jms-indexer-pluggable#112 (master - d7c3197 : Kevin S. Clarke): The build has errored.
[travis-ci] Change view : https://github.com/futures/fcrepo-jms-indexer-pluggable/compare/8ed3807fd402...d7c3197c14db
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo-jms-indexer-pluggable/builds/24553146
* travis-ci leaves
<pivotal-bot>Esme Cowles started "FedoraNodes unit tests" https://www.pivotaltracker.com/story/show/7064582413:12
Esme Cowles added comment: "@awoods Is the task here to determine how many files can be in a single directory? Or how many files total ..." https://www.pivotaltracker.com/story/show/7072457213:18
Andrew Woods added comment: "Total is the first priority.13:19
Having a recommended "max" for a single directory is a second." https://www.pivotaltracker.com/story/show/70724572
Andrew Woods deleted "new tickets below" https://www.pivotaltracker.com/story/show/7070062213:47
<ksclarke>cbeer: yes, I was using git history... problem was the file was renamed and lost git history
cbeer: or did you mean rather than adding a @author13:48
<cbeer>ksclarke; rather than @author. but, as i said, i'd never think to look at the javadocs to find out who worked on a file either.
* ksclarke nods13:49
* github-ff joins13:52
[fcrepo4] cbeer created delete-obj (+1 new commit): http://git.io/qBje1g
fcrepo4/delete-obj 67d8771 Chris Beer: NodeService#deleteObject should also clean up inbound references to a node
* github-ff leaves
<pivotal-bot>Chris Beer added comment: "https://github.com/futures/fcrepo4/pull/344" https://www.pivotaltracker.com/story/show/70772942
Chris Beer finished "NodeService#deleteObject should also clean up inbound references to the object" https://www.pivotaltracker.com/story/show/70772942
* github-ff joins
[fcrepo4] cbeer opened pull request #344: NodeService#deleteObject should also clean up inbound references to a node (master...delete-obj) http://git.io/HSXs8w
* github-ff leaves
* cjcolvar leaves14:02
<benpennell>should we be updating our license headers in new files to be 2014 now?14:09
<pivotal-bot>Andrew Woods added comment: "Looks good. Pending minor code review comments." https://www.pivotaltracker.com/story/show/6985953014:12
Andrew Woods rejected "Reduce build and runtime logging verbosity" https://www.pivotaltracker.com/story/show/69859530
* travis-ci joins14:13
[travis-ci] futures/fcrepo4#1901 (delete-obj - 67d8771 : Chris Beer): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/commit/67d87710b725
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24557754
* travis-ci leaves
<awoods>benpennell: https://github.com/futures/fcrepo4/pull/33914:14
benpennell: I will be pulling this in today.
<benpennell>yeah, that's why i was asking
<awoods>benpennell: yes, please use 2014.14:15
<benpennell>will do, thanks
<tecoripa>mikeAtUVa: quick question about this exception you catch and process in VersionServiceImpl:14:16
https://github.com/futures/fcrepo4/blob/master/fcrepo-kernel/src/main/java/org/fcrepo/kernel/services/VersionServiceImpl.java#L156
mikeAtUVa: it makes Sonar sad14:17
mikeAtUVa: is there another way to see if the version has a label or not?
<mikeAtUVa>tecoripa: that's a good question... I would assume not or otherwise that seems like a clumsy way to do it... I'll investigate...14:18
<tecoripa>mikeAtUVa: I'm okay with putting a LOGGER.debug statement there to shut Sonar up, but it would be nice to avoid the flow control.
mikeAtUVa: would this work? http://www.day.com/maven/jsr170/javadocs/jcr-1.0/javax/jcr/version/VersionHistory.html#hasVersionLabel%28java.lang.String%2914:19
<mikeAtUVa>tecoripa: eh, it looks like I could have gotten all the labels with getVersionLabels() and searched the array for the given label...
tecoripa: or that.
<tecoripa>mikeAtUVa: okay, I''ll refactor14:20
i.e., fix it
<mikeAtUVa>Feel free to fix it. Is there a lot of overhead in exception throwing and catching or is it just bad form?
* ermadmix joins14:22
<ksclarke>tecoripa was it you who said you knew of a tool to add javadoc @author tags based on git history?14:24
<tecoripa>ksclarke: "knew" is a bit strong... but I thought I had come acrosee something like that a few weeks ago14:26
across
<ksclarke>any recollection of what it's name might be?
to give me a hand up on searching
<tecoripa>ksclarke: nope
ksclarke: sorry14:27
<ksclarke>np :-)
<tecoripa>ksclarke: wonder how hard it would be to script, though... something to do a git log, parse the results, reformat it in javadoc
* ksclarke nods14:28
<tecoripa>mikeAtUVa: generally, using exceptions for flow control is frowned upon. Catch blocks should just throw or log, and do any kind of rollback/cleanup that the error may entail.
<mikeAtUVa>tecoripa: that sounds reasonable.14:29
<pivotal-bot>Andrew Woods added comment: "Looks good, pending code comment." https://www.pivotaltracker.com/story/show/6966621814:31
Andrew Woods rejected "Update code header copyright" https://www.pivotaltracker.com/story/show/69666218
* nikhiltri leaves14:33
<pivotal-bot>Benjamin Pennell added comment: "https://github.com/futures/fcrepo-module-auth-xacml/pull/1" https://www.pivotaltracker.com/story/show/7068813414:50
<tecoripa>mikeAtUVa: I think I'm figuring out why you took that VersionException route...14:53
<cbeer>escowles: i'm conflicted about references vs weakreferences.. especially after writing the code to remove references on DELETE, i'm not sure why we shouldn't just turn the fedorarelsext properties into WEAKREFERENCE and call it a day14:54
<mikeAtUVa>tecoripa: is there an issue?14:55
<escowles>cbeer: i'm not at all sure what the correct behavior is -- cascading deletes, failing with 412/409 status, or converting to weakrefs or URIs -- all seem to have some merit
<pivotal-bot>Mike Durbin started "Update code header copyright" https://www.pivotaltracker.com/story/show/69666218
Mike Durbin finished "Update code header copyright" https://www.pivotaltracker.com/story/show/69666218
<tecoripa>mikeAtUVa: I'm getting odd results when I replace it with code that use hasVersionLabel(). It makes two of your tests fail, but for the life of me, I can't figure out how the logical effect of my code differs from what you were doing.
<cbeer>escowles: the other weird thing in our _ref properties are all weakreferences anyway
<pivotal-bot>Mike Durbin added comment: "I pushed the changes..." https://www.pivotaltracker.com/story/show/6966621814:56
<cbeer>so i'm not sure why we (I?) chose REFERENCE for the rels-ext types
<mikeAtUVa>tecoripa: Oh... yeah... that's the joy of using mocks in unit tests.
<escowles>cbeer: cascading deletes would certainly make the test teardown easier, but i'm guessing most real repos would rather get an error if you tried to delete something with dangling inbound references
<tecoripa>mikeAtUVa: I can push my change to a branch, and put the test errors I'm seeing in a gist
<mikeAtUVa>tecoripa: in those tests we probably don't stub out responses to "hasVersionLabel".
<tecoripa>ahhhh... okay
<cbeer>escowles: i suppose by leaving them as REFERENCE, we have the option of introducing a parameter later to optionally delete the references14:57
<tecoripa>mikeAtUVa: so I should mock that, huh?
<escowles>i agree -- i thought the WEAKREFERENCE thing was a little strange, and couldn't really figure out why they were created or why they were causing errors, but found a way to find and handle them
<mikeAtUVa>tecoripa: for months we've been complaining about unit testing with mocks being especially fragile given our interactions with modeshape... I suspect you're encountering an example.
<escowles>cbeer: maybe we need to have a header to specify the delete behavior?14:58
<tecoripa>mikeAtUVa: okay, good to know. Is the accepted parctice to mock out everything, then? should I change the test?
<cbeer>escowles: yeah, i was starting to wonder if the Prefer: header was appropriate here too
although i think it's mainly about response content14:59
<escowles>DELETE /rest/foo HTTP/1.1^MPrefer: KILL_IT_WITH_FIRE
<mikeAtUVa>tecoripa: Everything? Mocking out every possible call would be essentially reimplementing modeshape... we typically mock all the calls that are invoked by our implementation... but as we've oft complained... that ties our tests to our implementation which defeats much of the purpose of unit testing (though it does improve our test coverage numbers).
<cbeer>Prefer: CascadeDeletes; predicate="xyz"
<escowles>cbeer: are we going to know which predicates are pointing at a node?15:00
<cbeer>escowles: yep, we know everything.
<tecoripa>mikeAtUVa: by "everything", I mean "everything that is used within a test"
<cbeer>(as long as you haven't used a uni-directional reference type, which i think MODE introduced recently..)
<escowles>cbeer: in that case, that makes perfect sense so you can delete some and leave others
<mikeAtUVa>tecoripa: yeah. that's typically what we do.
<tecoripa>mikeAtUVa: sounds good. I'll do that, then. thanks15:01
<mikeAtUVa>tecoripa: thanks for cleaning up my messy implementation and the tests:)
<cbeer>awoods: have you started to look at https://github.com/futures/fcrepo4/pull/340 at all? else, i'm going to force push some changes
<tecoripa>mikeAtUVa: np. glad to be of use15:02
<awoods>cbeer: nope. force it.
<cbeer>thx.
* github-ff joins
[fcrepo4] cbeer force-pushed inbound-references from ffab229 to 562664f: http://git.io/FJFZkA
fcrepo4/inbound-references 562664f Chris Beer: Optionally include inbound references in the object rdf serialization...
* github-ff leaves
<tecoripa>mikeAtUVa: adding the method to the mock in the test fixed it. thanks.15:07
* nikhiltri joins
* fcrepo-bot joins15:11
<pivotal-bot>Andrew Woods added comment: "Pending code comments." https://www.pivotaltracker.com/story/show/7059766615:13
Andrew Woods rejected "Getting lock metadata multiple times in a session throws exception." https://www.pivotaltracker.com/story/show/70597666
Longshou Situ added comment: "I've removed those slides in trouble, and wrapped up the slide and paragraphs to fix the messy displaying ..." https://www.pivotaltracker.com/story/show/7067361215:15
Longshou Situ finished "Clean-up Fedora wiki main page" https://www.pivotaltracker.com/story/show/70673612
* github-ff joins15:21
[fcrepo4] awoods pushed 1 new commit to master: http://git.io/eIxcaQ
fcrepo4/master cef5926 Mike Durbin: Updated license copyright year to 2014....
* github-ff leaves
<pivotal-bot>Andrew Woods added comment: "Resolved with: https://github.com/futures/fcrepo4/commit/cef5926859f7fd9b8763042b5dc43905085f07ed" https://www.pivotaltracker.com/story/show/69666218
Andrew Woods delivered "Update code header copyright" https://www.pivotaltracker.com/story/show/6966621815:22
* github-ff joins
[fcrepo4] awoods closed pull request #339: Updated license copyright year to 2014. (master...license-update) http://git.io/H9PiVw
* github-ff leaves
* travis-ci joins15:25
[travis-ci] futures/fcrepo4#1904 (inbound-references - 562664f : Chris Beer): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/ffab229105d0...562664f0db43
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24563061
* travis-ci leaves
* ksclarke bangs head against desk15:36
* travis-ci joins15:41
[travis-ci] futures/fcrepo4#1906 (master - cef5926 : Mike Durbin): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/e592293cf549...cef5926859f7
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24564707
* travis-ci leaves
<awoods>ksclarke: everything ok?15:42
<ksclarke>awoods: just frustrated... eclipse has a line wrap bug that doesn't recognize a line needs to be wrapped if last character is a semicolon, differences in checkstyle and eclipse indentations, etc.15:44
<awoods>ksclarke: hang in there.15:45
<ksclarke>:-)15:46
* github-ff joins15:58
[fcrepo4] escowles created fedora-nodes-tests (+1 new commit): http://git.io/JJEKzw
fcrepo4/fedora-nodes-tests addcd5d Esmé Cowles: Adding unit tests for new/refactored HEAD, OPTIONS, and POST methods of FedoraNodes
* github-ff leaves
<pivotal-bot>Andrew Woods accepted "Clean-up Fedora wiki main page" https://www.pivotaltracker.com/story/show/7067361215:59
* github-ff joins16:01
[fcrepo4] escowles opened pull request #345: Adding unit tests for new/refactored HEAD, OPTIONS, and POST methods of FedoraNodes (master...fedora-nodes-tests) http://git.io/P2W2jg
* github-ff leaves
<pivotal-bot>Andrew Woods added comment: "https://github.com/futures/fcrepo4/pull/343" https://www.pivotaltracker.com/story/show/6601468616:05
* travis-ci joins16:16
[travis-ci] futures/fcrepo4#1907 (fedora-nodes-tests - addcd5d : Esmé Cowles): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/commit/addcd5d6d942
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24568019
* travis-ci leaves
* dwilcox leaves
* dwilcox joins16:17
* fcrepo-bot leaves16:21
* dwilcox leaves16:26
<pivotal-bot>Esme Cowles added comment: "https://github.com/futures/fcrepo4/pull/345" https://www.pivotaltracker.com/story/show/7064582416:32
Esme Cowles finished "FedoraNodes unit tests" https://www.pivotaltracker.com/story/show/7064582416:33
* github-ff joins16:38
[fcrepo4] awoods pushed 1 new commit to master: http://git.io/aCnWLg
fcrepo4/master 48d02cc Kai Sternad: Integration test to ensure that transactions cannot be hijacked...
* github-ff leaves
<awoods>escowles: ping16:39
<pivotal-bot>Andrew Woods added comment: "Resolved with: https://github.com/futures/fcrepo4/commit/48d02ccdd154b278cca80a5f45ea879ce21784a6" https://www.pivotaltracker.com/story/show/6601468616:40
<escowles>awoods: yes?
<awoods>escowles: In the git comment...
escowles: the logger "name" does not match the logger "level"
escowles: the level should be: "org.fcrepo.kernel.observer"16:41
escowles: hence the comment: "The level is probably a cut and paste error."
<escowles>awoods: oh, i see what you mean now -- yes a cut and paste error... will fix it now
* github-ff joins16:44
[fcrepo4] awoods closed pull request #343: Fcrepo 66014686 (master...fcrepo-66014686) http://git.io/1NrYBQ
* github-ff leaves
<pivotal-bot>Andrew Woods delivered "Write an integration test to ensure that transactions cannot be hijacked." https://www.pivotaltracker.com/story/show/66014686
* github-ff joins
[fcrepo4] escowles pushed 1 new commit to reduced-logging: http://git.io/jrXi0w
fcrepo4/reduced-logging ddcdf4b Esmé Cowles: Fixing org.fcrepo.kernel.observer error, restoring repo properties logging to info
* github-ff leaves
<pivotal-bot>Esme Cowles started "Reduce build and runtime logging verbosity" https://www.pivotaltracker.com/story/show/6985953016:45
Esme Cowles added comment: "I've addressed the comments with an additional commit in https://github.com/futures/fcrepo4/pull/342" https://www.pivotaltracker.com/story/show/69859530
Esme Cowles finished "Reduce build and runtime logging verbosity" https://www.pivotaltracker.com/story/show/69859530
* mikeAtUVa leaves16:51
* github-ff joins16:54
[fcrepo4] awoods pushed 2 new commits to master: http://git.io/8N9RaQ
fcrepo4/master 6709914 Esmé Cowles: Configuring logging to INFO level by default, reducing error messages and stacktraces, changing handled exception logging to info instead of warn
fcrepo4/master 38482a1 Esmé Cowles: Standardizing logging levels and making them configurable with system properties...
* github-ff leaves
<pivotal-bot>Andrew Woods added comment: "Resolved with: https://github.com/futures/fcrepo4/compare/48d02ccdd154...38482a1ebfec" https://www.pivotaltracker.com/story/show/69859530
Andrew Woods delivered "Reduce build and runtime logging verbosity" https://www.pivotaltracker.com/story/show/6985953016:55
* github-ff joins
[fcrepo4] awoods deleted reduced-logging at ddcdf4b: http://git.io/AGQ6LQ
* github-ff leaves
* travis-ci joins16:56
[travis-ci] futures/fcrepo4#1909 (master - 48d02cc : Kai Sternad): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/cef5926859f7...48d02ccdd154
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24571263
* travis-ci leaves
<awoods>escowles: why did you change the jena version back to 2.11.0?16:58
<escowles>awoods: because i got a convergent dependency error with 2.11.1
<awoods>escowles: I had a similar issue on the indexer side, but resolved it: https://github.com/futures/fcrepo-jms-indexer-pluggable/commit/8ed3807fd4022ad2910c265b513711c3b2f0ad9116:59
escowles: that was today
<escowles>i'll run the build with 2.11.1 again and figure out what needs to change to work with 2.11.117:00
<awoods>escowles: I believe the resolution had to do with jena-arq: https://github.com/futures/fcrepo-jms-indexer-pluggable/commit/8ed3807fd4022ad2910c265b513711c3b2f0ad91#diff-600376dffeb79835ede4a0b285078036R22317:01
* scossu leaves
<cbeer>awoods: can i bug you to review https://github.com/futures/fcrepo4/pull/340 and https://github.com/futures/fcrepo4/pull/344 soon?
<awoods>cbeer: sure, those are next17:02
<pivotal-bot>Andrew Woods added comment: "https://github.com/futures/fcrepo4/pull/340" https://www.pivotaltracker.com/story/show/7072328417:04
Andrew Woods edited "Include inbound references in the RDF serialization of a node" https://www.pivotaltracker.com/story/show/70723284
* travis-ci joins17:05
[travis-ci] futures/fcrepo4#1910 (reduced-logging - ddcdf4b : Esmé Cowles): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/5d9656cd1c13...ddcdf4bbb976
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24571774
* travis-ci leaves
* travis-ci joins17:14
[travis-ci] futures/fcrepo4#1912 (master - 38482a1 : Esmé Cowles): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/48d02ccdd154...38482a1ebfec
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24572599
* travis-ci leaves
* github-ff joins
[fcrepo4] escowles pushed 1 new commit to fedora-nodes-tests: http://git.io/S7ldAA
fcrepo4/fedora-nodes-tests ae920a1 Esmé Cowles: Switching back to Jena 2.11.1 -- convergent dependency error did not happen again
* github-ff leaves
* scossu joins17:15
<pivotal-bot>Esme Cowles added comment: "@awoods: I tried a clean build with Jena 2.11.1 and didn't see the convergent dependency error, so I've push..." https://www.pivotaltracker.com/story/show/70645824
* travis-ci joins17:36
[travis-ci] futures/fcrepo4#1913 (fedora-nodes-tests - ae920a1 : Esmé Cowles): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/addcd5d6d942...ae920a129703
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24574506
* travis-ci leaves
* scossu leaves17:46
* github-ff joins17:53
[fcrepo4] awoods pushed 1 new commit to inbound-references: http://git.io/YIsVXA
fcrepo4/inbound-references 4d7292f Andrew Woods: Minor code review updates
* github-ff leaves
* github-ff joins17:54
[fcrepo4] awoods pushed 1 new commit to master: http://git.io/KSpFow
fcrepo4/master d77194c Chris Beer: Optionally include inbound references in the object rdf serialization...
* github-ff leaves
* github-ff joins17:55
[fcrepo4] awoods deleted inbound-references at 4d7292f: http://git.io/rKiRvw
* github-ff leaves
<pivotal-bot>Andrew Woods added comment: "Resolved with: https://github.com/futures/fcrepo4/commit/d77194ca77f9eb67cf735fe71d1e30c0d34fa033" https://www.pivotaltracker.com/story/show/70723284
Andrew Woods delivered "Include inbound references in the RDF serialization of a node" https://www.pivotaltracker.com/story/show/70723284
* travis-ci joins17:57
[travis-ci] futures/fcrepo4#1915 (inbound-references - 4d7292f : Andrew Woods): The build has errored.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/562664f0db43...4d7292f9bdb5
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24577266
* travis-ci leaves
* nikhiltri leaves18:02
* nikhiltri joins18:05
* scossu joins18:06
* scossu1 joins18:09
* nikhiltri leaves
* scossu leaves18:11
* travis-ci joins18:15
[travis-ci] futures/fcrepo4#1916 (master - d77194c : Chris Beer): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/38482a1ebfec...d77194ca77f9
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24577388
* travis-ci leaves
* ksclarke leaves18:17
* github-ff joins18:29
[fcrepo4] awoods pushed 1 new commit to delete-obj: http://git.io/1f038A
fcrepo4/delete-obj 2551f17 Andrew Woods: Minor code review update
* github-ff leaves
* github-ff joins18:31
[fcrepo4] awoods pushed 1 new commit to master: http://git.io/chzWMA
fcrepo4/master ed41237 Chris Beer: NodeService#deleteObject should also clean up inbound references to a node...
* github-ff leaves
<pivotal-bot>Andrew Woods added comment: "Resolved with: https://github.com/futures/fcrepo4/commit/ed41237e2ee9d7d5cec4c8dad6d599f30230c5db" https://www.pivotaltracker.com/story/show/70772942
Andrew Woods delivered "NodeService#deleteObject should also clean up inbound references to the object" https://www.pivotaltracker.com/story/show/70772942
* ermadmix leaves18:32
* github-ff joins
[fcrepo4] awoods deleted delete-obj at 2551f17: http://git.io/DzP81w
* github-ff leaves
<pivotal-bot>Andrew Woods added comment: "Same with @benjaminpennell " https://www.pivotaltracker.com/story/show/7067342618:34
Andrew Woods added comment: "@scottprater, can you try this on your Windows machine this week?" https://www.pivotaltracker.com/story/show/7067342618:35
* github-ff joins
[fcrepo4] sprater opened pull request #346: Fcrepo 70584818 (master...fcrepo-70584818) http://git.io/lm-unQ
* github-ff leaves
<pivotal-bot>Scott Prater added comment: "https://github.com/futures/fcrepo4/pull/346" https://www.pivotaltracker.com/story/show/7058481818:37
Scott Prater finished "Address "Critical" Sonar Issues" https://www.pivotaltracker.com/story/show/70584818
* travis-ci joins
[travis-ci] futures/fcrepo4#1917 (delete-obj - 2551f17 : Andrew Woods): The build has errored.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/67d87710b725...2551f1717a33
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24579962
* travis-ci leaves
* ermadmix joins18:38
<pivotal-bot>Scott Prater added comment: "@awoods: will do." https://www.pivotaltracker.com/story/show/7067342618:39
* tecoripa leaves18:41
* travis-ci joins18:46
[travis-ci] futures/fcrepo4#1918 (master - ed41237 : Chris Beer): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/d77194ca77f9...ed41237e2ee9
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24580050
* travis-ci leaves
<awoods>cbeer/escowles: Is it safe to assume that this update can be removed:18:50
https://github.com/futures/fcrepo4/commit/9183bc18b1934ba89bfba908135689825367b78c#diff-3cd9c4d0ac7b48b76f2689212b21bc44R682
given: https://github.com/futures/fcrepo4/commit/ed41237e2ee9d7d5cec4c8dad6d599f30230c5db18:51
<cbeer>awoods: i don't know, although it certainly isn't safe until we can use MODE 3.7.3
<awoods>cbeer: your latest update removes incoming references
cbeer: the exception in the diff catches the case when there still exist incoming references.18:52
cbeer: where does ModeShape 3.7.3 come in?
<cbeer>awoods: 3.7.2 fixes a bug in the reference management. until we can use it (which we can't, because of a bug that will be fixed in 3.7.3), ReferentialIntegrityExceptions are still throw18:54
(for the wrong reasons, but..)
<awoods>cbeer: thanks18:55
<cbeer>and, we might want to keep that code anyway. presumably with access controls or something, you might not have permission to remove some of those references
* github-ff joins19:00
[fcrepo4] awoods pushed 1 new commit to master: http://git.io/RciIAg
fcrepo4/master 92ce4ca Esmé Cowles: Add unit tests for new/refactored HEAD, OPTIONS, and POST methods of FedoraNodes...
* github-ff leaves
<awoods>cbeer: that is reasonable
<cbeer>maybe, maybe not.19:01
<pivotal-bot>Andrew Woods added comment: "Resolved with: https://github.com/futures/fcrepo4/commit/92ce4ca0156c12dfa2f9190dc00b8968da4964f0" https://www.pivotaltracker.com/story/show/70645824
* github-ff joins
[fcrepo4] awoods closed pull request #345: Adding unit tests for new/refactored HEAD, OPTIONS, and POST methods of FedoraNodes (master...fedora-nodes-tests) http://git.io/P2W2jg
* github-ff leaves
<pivotal-bot>Andrew Woods delivered "FedoraNodes unit tests" https://www.pivotaltracker.com/story/show/70645824
Andrew Woods accepted "FedoraNodes unit tests" https://www.pivotaltracker.com/story/show/7064582419:09
Andrew Woods accepted "NodeService#deleteObject should also clean up inbound references to the object" https://www.pivotaltracker.com/story/show/70772942
Andrew Woods accepted "Update checkstyles for fcrepo-jms-indexer-pluggable" https://www.pivotaltracker.com/story/show/7064805019:10
Andrew Woods accepted "Include inbound references in the RDF serialization of a node" https://www.pivotaltracker.com/story/show/70723284
Andrew Woods accepted "Bump dependency versions in fcrepo-indexer-jms-pluggable" https://www.pivotaltracker.com/story/show/70774640
Andrew Woods accepted "Reduce build and runtime logging verbosity" https://www.pivotaltracker.com/story/show/69859530
Andrew Woods accepted "Update code header copyright" https://www.pivotaltracker.com/story/show/69666218
Andrew Woods accepted "Update upstream dependencies" https://www.pivotaltracker.com/story/show/70676928
Andrew Woods accepted "Write an integration test to ensure that transactions cannot be hijacked." https://www.pivotaltracker.com/story/show/66014686
* ermadmix leaves
* travis-ci joins19:15
[travis-ci] futures/fcrepo4#1920 (master - 92ce4ca : Esmé Cowles): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/ed41237e2ee9...92ce4ca0156c
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24581783
* travis-ci leaves
* scossu1 leaves19:17
* edInCo leaves19:36
* ermadmix joins19:41
* github-ff joins19:47
[fcrepo4] lsitu opened pull request #347: Improve and simplify fcr:versions response triples to have links only. (master...feature/versions) http://git.io/R5GmPw
* github-ff leaves
<pivotal-bot>Longshou Situ added comment: "https://github.com/futures/fcrepo4/pull/347" https://www.pivotaltracker.com/story/show/7047505819:48
Longshou Situ finished "Improve and simplify fcr:versions response triples." https://www.pivotaltracker.com/story/show/7047505819:49
* ermadmix leaves20:15
* github-ff joins20:34
[fcrepo4] cbeer force-pushed refactor-fixity from fc0c981 to 5095b93: http://git.io/TbjnLg
fcrepo4/refactor-fixity 3c6efb5 Chris Beer: Update NodeRdfContext to be more defensive about missing type information
fcrepo4/refactor-fixity 2cc696d Chris Beer: Accept null values for comparing against a calculated checksum
fcrepo4/refactor-fixity 822170f Chris Beer: Remove the self-healing-after-fixity-check-failure feature...
* github-ff leaves
* ermadmix joins20:40
* travis-ci joins
[travis-ci] futures/fcrepo4#1922 (refactor-fixity - 5095b93 : Chris Beer): The build has errored.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/fc0c9812d5cb...5095b93367e5
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24586290
* travis-ci leaves
* longshou leaves20:49
* github-ff joins21:11
[fcrepo4] cbeer created 404-debug (+1 new commit): http://git.io/jiDiMg
fcrepo4/404-debug 81c3df9 Chris Beer: PathNotFoundExceptionMapper should use the DEBUG log level
* github-ff leaves
* ermadmix leaves21:13
* github-ff joins
[fcrepo4] cbeer created log-levels (+1 new commit): http://git.io/NvQUlA
fcrepo4/log-levels 0de30c8 Chris Beer: Reduce log level for "normal" HTTP API operations
* github-ff leaves
* github-ff joins
[fcrepo4] cbeer opened pull request #348: Reduce log level for "normal" HTTP API operations (master...log-levels) http://git.io/8WaS7Q
* github-ff leaves
* travis-ci joins21:14
[travis-ci] futures/fcrepo4#1923 (404-debug - 81c3df9 : Chris Beer): The build has errored.
[travis-ci] Change view : https://github.com/futures/fcrepo4/commit/81c3df9b555f
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24587772
* travis-ci leaves
<pivotal-bot>Chris Beer added "Reduce log level for normal operation messages" https://www.pivotaltracker.com/story/show/70816006
Chris Beer started "Reduce log level for normal operation messages" https://www.pivotaltracker.com/story/show/70816006
Chris Beer finished "Reduce log level for normal operation messages" https://www.pivotaltracker.com/story/show/70816006
* travis-ci joins21:27
[travis-ci] futures/fcrepo4#1924 (log-levels - 0de30c8 : Chris Beer): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/commit/0de30c88ded0
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24587845
* travis-ci leaves
* ksclarke joins21:28
* ermadmix joins21:54
* ermadmix leaves22:27
* ermadmix joins22:36
* github-ff joins22:47
[fcrepo4] awoods pushed 1 new commit to master: http://git.io/t015Ug
fcrepo4/master f2057e1 Andrew Woods: Merge pull request #348 from futures/log-levels...
* github-ff leaves
<pivotal-bot>Andrew Woods accepted "Reduce log level for normal operation messages" https://www.pivotaltracker.com/story/show/70816006
Stefano Cossu added "Missing slash character in Fedora base URL in JMS indexer" https://www.pivotaltracker.com/story/show/7081968022:55
* travis-ci joins23:00
[travis-ci] futures/fcrepo4#1926 (master - f2057e1 : Andrew Woods): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/92ce4ca0156c...f2057e10a067
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/24591211
* travis-ci leaves
* ermadmix leaves23:08
* ermadmix joins23:30

Generated by Sualtam