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

Using timezone: Eastern Standard Time
* nbanks leaves05:04
* nbanks joins05:08
* kaarefc leaves05:16
* kaarefc joins06:48
* fcrepo-bot joins07:38
* mikeAtUVa leaves08:27
* fcrepo-bot leaves08:40
* awoods joins08:42
<pivotal-bot____>Mike Durbin added "Expose paging over large federation of fedora 3 content." https://www.pivotaltracker.com/story/show/5642959408:53
* mikeAtUVa joins
<pivotal-bot____>Mike Durbin added "When federating over a fedora 3 repository, ensure that datastreams are accessible." https://www.pivotaltracker.com/story/show/5642986208:58
Mike Durbin edited "When federating over a fedora 3 repository, ensure that datastreams are accessible." https://www.pivotaltracker.com/story/show/56429862
* kaarefc leaves09:10
* jay joins
* gregjansen joins09:13
* kaarefc joins09:32
* awoods leaves09:34
* awoods joins09:41
* kaarefc leaves09:45
<pivotal-bot____>Andrew Woods delivered "Backup a running repository" https://www.pivotaltracker.com/story/show/4610822909:47
Andrew Woods accepted "Backup a running repository" https://www.pivotaltracker.com/story/show/46108229
* kaarefc joins
<pivotal-bot____>Andrew Woods edited "Expose paging over large federation of fedora 3 content." https://www.pivotaltracker.com/story/show/5642959409:48
Andrew Woods added "Document Backup/Restore Process" https://www.pivotaltracker.com/story/show/5643319009:50
Andrew Woods started "Document Backup/Restore Process" https://www.pivotaltracker.com/story/show/56433190
* kaarefc leaves09:53
* kaarefc joins09:54
<awoods>mikeAtUVa: Do you think your new issue re:F3 Datastreams being accessible has to do with the fact that you are still using jcr:content instead of fcr:content in ID.java?10:00
* mikeAtUVa leaves10:01
* mikeAtUVa joins
* kaarefc leaves
* kaarefc joins
<mikeAtUVa>awoods: when I changed that to fcr:content I get exceptions because the fcr namespace isn't registered.10:02
awoods: as it is, the UI presents a link to the fcr:content which, while not throwing an exception, fails to deliver the content
awoods: when I take up that task, I'll benefit from setting up a trace or at least some more complete logging of the relevant bits. It's likely I've improperly set (or omitted) properties on the nodes I'm representing as fedora objects.10:05
* nbanks leaves10:06
<awoods>mikeAtUVa: The jcr/fcr issue came to mind as a possible culprit. But if you have already thought of that, we can resolve that issue when we (you) get to it.10:07
* nbanks joins10:09
* kaarefc leaves10:13
* kaarefc joins
* kaarefc leaves10:19
<pivotal-bot____>Andrew Woods added "Fedora3 Connector Integration Tests" https://www.pivotaltracker.com/story/show/5643681410:22
Andrew Woods edited "Fedora3 Connector Integration Tests" https://www.pivotaltracker.com/story/show/5643681410:23
* github-ff joins10:24
[fcrepo-fedora3-federation-connector] awoods pushed 1 new commit to master: http://git.io/52nC5g
fcrepo-fedora3-federation-connector/master 6736341 Andrew Woods: Add travis build file
* github-ff leaves
<pivotal-bot____>Andrew Woods accepted "Write a proof of concept read-only implementation of a federation over fedora 3 content" https://www.pivotaltracker.com/story/show/5576913210:25
Andrew Woods edited "Write PEP to enforce access policies around modeshape operations." https://www.pivotaltracker.com/story/show/55891634
<awoods>gregjansen: Is there a pull request associated with https://www.pivotaltracker.com/story/show/5589163410:26
<pivotal-bot____>feature: Write PEP to enforce access policies around modeshape operations. (finished) / owner: Gregory Jansen
Andrew Woods added comment: "Is there a pull-request or commit associated with this?" https://www.pivotaltracker.com/story/show/5589163410:31
* travis-ci joins10:32
[travis-ci] futures/fcrepo-fedora3-federation-connector#1 (master - 6736341 : Andrew Woods): The build has errored.
[travis-ci] Change view : https://github.com/futures/fcrepo-fedora3-federation-connector/compare/5b731489c47b...673634170c95
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo-fedora3-federation-connector/builds/11018733
* travis-ci leaves
* edInCo joins10:52
<awoods>All: Reminder that today's committer call is on ReadyTalk: https://wiki.duraspace.org/display/FCREPO/2013-09-05+-+Fedora+Committer+Meeting10:56
* gregjansen1 joins10:59
* gregjansen leaves11:00
<gregjansen1>what is the committer call conference line?11:02
<awoods>U.S.A/Canada toll free: 866-740-1260, participant code: 2257295
<pivotal-bot____>Gregory Jansen added comment: "https://github.com/futures/fcrepo4/pull/109" https://www.pivotaltracker.com/story/show/5589163411:05
<nbanks>better late than never.11:17
<pivotal-bot____>Osman Din added comment: "Change to resolve the issue mentioned in the previous comment: ""11:40
https://github.com/osmandin/fcrepo-jms-indexer..." https://www.pivotaltracker.com/story/show/55768870
* barmintor joins11:53
* edInCo leaves12:01
* osmandin joins12:10
* jay leaves12:24
* osmandin leaves12:28
* edInCo joins12:43
* github-ff joins
[fcrepo-fedora3-federation-connector] awoods pushed 1 new commit to master: http://git.io/JGkvbg
fcrepo-fedora3-federation-connector/master ef50b84 Andrew Woods: Add Sonatype snapshot repository to pom
* github-ff leaves
* travis-ci joins12:50
[travis-ci] futures/fcrepo-fedora3-federation-connector#2 (master - ef50b84 : Andrew Woods): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo-fedora3-federation-connector/compare/673634170c95...ef50b84ada9b
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo-fedora3-federation-connector/builds/11024247
* travis-ci leaves
* gregjansen1 leaves12:51
* gregjansen joins13:04
<awoods>gregjansen: It looks like your PR has checkstyle errors.13:07
* jay joins13:25
<awoods>mikeAtUVa: In thinking more about the software delivery process, if there is a fair likelihood that there will be code-review comments, adding a link in Pivotal to a branch on your personal git-repo makes sense. That way, reviewers can add comments to commits and work can continue on that branch until the review iteration has completed. Then the developer can squash commits and submit a PR, or I can do that from my side.13:26
<gregjansen>awoods: fixing them13:47
<awoods>gregjansen: As I mentioned to mikeAtUVa, feel free to just keep working in your personal git-repo... I can review from there.13:51
bbi ~15
<gregjansen>awoods: wrangling my eclipse settings and having to do a lot of ctrl-z to save formatting intact. I see most of these checkstyle issues in master too.14:23
awoods: actually most, if not all, warnings in futures/fcrepo master are about line length. The files I've editing seem to have additional indention issues..14:25
awoods: nm, I see that the line length issue is currently just a warning and not a blocker14:30
<mikeAtUVa>So, I found the problem that prevents me from exposing a fedora 3 repository object and allowing access to all objects....14:32
Several methods in JcrRdfTools that get an RDF model for a node include properties of all of its immediate children. This is incompatible with having any object that has "a lot" of children.14:35
<gregjansen>mikeAtUVa: so that's an issue b/c they all end up in an objects node? (i.e. they have 1 big parent)14:39
<mikeAtUVa>Yeah, so I was trying to view the node for the repository, and wondering why every page of children is being fetched....14:40
<gregjansen>oh god
that could be problematic
not just one page in the connector sense either, b/c I'm guessing our JcrRdfTools just keep going14:41
<mikeAtUVa>Within JcrRdfTools.addJcrPropertiesToModel, which is eventually invoked by FedoraNodes.describe there's a call to node.hasNode(), which is the culpret.
That single call could likely be smartened up since it's only looking for content, and we could do a preliminary check to only look for content on nodes that would likely have content (datastreams)...14:42
But my changes break a bunch of tests and I'm not well-versed enough to determine the full implications.14:43
Anywho... I'll keep digging:)14:45
<gregjansen>Is there a way to get it more optimistically and catch a not found exception?14:46
<awoods>mikeAtUVa: Sounds like a good start at bringing some sanity to the table.
<gregjansen>maybe try getNode instead of hasNode..14:47
<awoods>mikeAtUVa: Is a paging approach not adequate?
<gregjansen>awoods: checkstyle issues other than line length are fixed. Cannot get line wrapping to happen correctly for string literals and comments. I suspect others have the same issue in eclipse.14:48
<mikeAtUVa>awoods: I can't get far enough to see, because when I expose all the content as pages, JcrRdfTools.addJcrPropertiesToModel() kills the application requesting every page first.
<awoods>gregjansen: As long as the build passing the style rules for now...
<gregjansen>awoods: k, that is done14:49
<awoods>gregjansen: got it: https://github.com/gregjan/fcrepo4/commit/b94be139c249c52925b948983337af5f8de36a06
mikeAtUVa: ?? do you know why JcrRdfTools.ajptm() kills the app?14:50
<mikeAtUVa>awoods: Yeah, because it invokes hasNode(), which in the worst case (where the node it's looking for doesn't exist) iterates over every child, which for my fedora repository ends up being several million objects... which *might* succeed in time, but clearly isn't desired behavior.14:52
<gregjansen>mikeAtUVa, awoods: Is this b/c the connector is implementing that pageable interface? Might we have better luck w/o that one?14:53
<mikeAtUVa>gregjansen: if I don't implement pageable, then I'm expected to add every child to the node from the start... which is what pageable is trying to get me around..14:54
I think I have a fix, if only I can install and test it without these checkstyle errors:P14:55
<mikeAtUVa>gregjansen: thanks, I was looking for that!
* ksclarke joins14:57
* mikeAtUVa leaves15:03
<awoods>gregjansen, mikeAtUVa: actually, for maven flags, "true" is implied. So you can use -Dcheckstyle.skip15:10
<gregjansen>awoods: ah, well that'll help with -DskipTests as well awoods++15:14
<awoods>gregjansen: indeed
* mikeAtUVa joins15:17
* nbanks leaves15:18
* nbanks joins15:19
<awoods>gregjansen: In looking at your commits, there appear to be lots of non-functional whitespace updates.15:20
gregjansen: Instead of reformatting entire files, you can highlight the code that you change/add and only format that (ctl F, or something like that)15:21
* nbanks leaves15:24
<gregjansen>In the recent spate of checkstyle fumbling I change eclipse to only format the code I've edited15:29
* nbanks joins15:42
* mikeAtUVa leaves16:00
<awoods>mikeAtUVa, gregjansen: https://wiki.duraspace.org/display/FF/2013-09-05+FF+Tech+Mtg
* mikeAtUVa joins16:04
<pivotal-bot____>Mike Durbin started "Expose paging over large federation of fedora 3 content." https://www.pivotaltracker.com/story/show/5642959416:10
* nbanks leaves16:27
* ksclarke leaves16:28
* edInCo leaves16:49
* nbanks joins16:50
* nbanks leaves16:55
<pivotal-bot____>Gregory Jansen added "Document PEP extension point" https://www.pivotaltracker.com/story/show/5647025217:06
Gregory Jansen edited "Document PEP extension point" https://www.pivotaltracker.com/story/show/5647025217:07
Gregory Jansen estimated "Document PEP extension point" as 2 points https://www.pivotaltracker.com/story/show/56470252
Gregory Jansen edited "Document PEP extension point" https://www.pivotaltracker.com/story/show/5647025217:08
Andrew Woods edited "Create a pull request to ModeShape adding the FileSystemMonitor class to their FileSystemConnector code." https://www.pivotaltracker.com/story/show/55985334
Andrew Woods unstarted "Create a pull request to ModeShape adding the FileSystemMonitor class to their FileSystemConnector code." https://www.pivotaltracker.com/story/show/55985334
Gregory Jansen started "Document PEP extension point" https://www.pivotaltracker.com/story/show/56470252
Andrew Woods edited "Create a pull request to ModeShape adding the FileSystemMonitor class to their FileSystemConnector code." https://www.pivotaltracker.com/story/show/55985334
Mike Durbin started "Create test rigging for a fedora 3 repository to support integration tests." https://www.pivotaltracker.com/story/show/5613537617:12
Mike Durbin estimated "Create test rigging for a fedora 3 repository to support integration tests." as 2 points https://www.pivotaltracker.com/story/show/5613537617:14
* nbanks joins17:20
* gregjansen leaves17:23
* jay leaves17:25
* nbanks leaves17:26
* nbanks joins17:52
* nbanks leaves17:56
* jay joins18:09
* jay leaves18:16
* nbanks joins18:22
* nbanks leaves18:27
<pivotal-bot____>Andrew Woods started "Fix IndexerGroup:113 Issue" https://www.pivotaltracker.com/story/show/5576887018:32
* nbanks joins18:52
* nbanks leaves18:57
* nbanks joins19:52
* nbanks leaves19:58
* ksclarke joins20:52
* nbanks joins20:53
* nbanks leaves20:58
* ksclarke leaves21:19
* github-ff joins21:46
[fcrepo4] awoods pushed 1 new commit to master: http://git.io/G_ynaQ
fcrepo4/master 19ff09e Gregory Jansen: Initial implementation of PEP...
* github-ff leaves
<pivotal-bot____>Andrew Woods added comment: "Resolved with: http://git.io/G_ynaQ21:48
Additional cleanup changes in attachment" https://www.pivotaltracker.com/story/show/55891634
Andrew Woods delivered "Write PEP to enforce access policies around modeshape operations." https://www.pivotaltracker.com/story/show/55891634
Andrew Woods finished "Fix IndexerGroup:113 Issue" https://www.pivotaltracker.com/story/show/5576887021:50
* nbanks joins21:53
* nbanks leaves21:58
* travis-ci joins21:59
[travis-ci] futures/fcrepo4#978 (master - 19ff09e : Gregory Jansen): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/2bdb4b5fc8a0...19ff09ef604a
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/11042117
* travis-ci leaves
* nbanks joins22:54
* nbanks leaves22:59
<bljenkins>Project fcrepo-fixity-corrupter build #246: SUCCESS in 6 min 13 sec: http://ci.fcrepo.org/jenkins/job/fcrepo-fixity-corrupter/246/23:24
Project fcrepo-legacy-api build #55: UNSTABLE in 14 min: http://ci.fcrepo.org/jenkins/job/fcrepo-legacy-api/55/23:32
Project fcrepo-jms-indexer-pluggable build #11: UNSTABLE in 19 min: http://ci.fcrepo.org/jenkins/job/fcrepo-jms-indexer-pluggable/11/23:37
* nbanks joins23:54
* nbanks leaves23:58
* nbanks joins00:53
* kaarefc joins00:55
* kaarefc leaves01:00
* kaarefc joins01:03
* kaarefc leaves01:24

Generated by Sualtam