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

Using timezone: Eastern Standard Time
* benpennell1 leaves01:54
* tecoripa joins08:07
* tecoripa leaves08:14
<pivotal-bot>Scott Prater accepted "Create XACML policies that enforce basic roles of reader, writer and admin" https://www.pivotaltracker.com/story/show/7068896208:17
Esme Cowles started "Largefile byte offset read" https://www.pivotaltracker.com/story/show/7123629208:18
* mikeAtUVa joins08:24
* benpennell1 joins08:25
* github-ff joins08:27
[fcrepo4] escowles created large-file-range (+1 new commit): http://git.io/y3jr1w
fcrepo4/large-file-range c230e7d Esmé Cowles: Updating range retrieval test to have configurable size to allow testing of large files by setting a system property (fcrepo.rangetest.skip)
* github-ff leaves
* github-ff joins08:29
[fcrepo4] escowles opened pull request #356: Updating range retrieval test to have configurable file size (master...large-file-range) http://git.io/qxSVtA
* github-ff leaves
<pivotal-bot>Esme Cowles added comment: "I've updated the range retrieval test to have a configurable file size: ""08:31
https://github.com/futures/fcrepo4/..." https://www.pivotaltracker.com/story/show/71236292
* dwilcox joins08:39
* benpennell1 leaves08:41
* travis-ci joins08:45
[travis-ci] futures/fcrepo4#1953 (large-file-range - c230e7d : Esmé Cowles): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/commit/c230e7db8acc
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/25150846
* travis-ci leaves
* ksclarke___ joins08:54
<pivotal-bot>Andrew Woods added comment: "See: #7123423809:05
Also see: https://wiki.duraspace.org/pages/diffpagesbyversion.action?pageId=40076971&select..." https://www.pivotaltracker.com/story/show/69444336
Andrew Woods accepted "Run integration tests with Filesystem Federation configured" https://www.pivotaltracker.com/story/show/69444336
Andrew Woods delivered "Run integration tests with Filesystem Federation configured" https://www.pivotaltracker.com/story/show/69444336
* awoods joins09:06
<pivotal-bot>Andrew Woods accepted "Add integration tests to verify Accept headers" https://www.pivotaltracker.com/story/show/7123475209:08
Andrew Woods accepted "Missing slash character in Fedora base URL in JMS indexer" https://www.pivotaltracker.com/story/show/70819680
* gregjansen joins09:17
So, question for the group.. We have discussed many times the possibility of attaching top-level defaults for things like role assignments and policies to the root node. It makes sense, they inherit from there. However, our workspace root node is not a fedora object and SPARQL update does not seem to work at the root, which is how folks would expect to modify the settings. What should we do?09:21
* tecoripa1 joins09:25
<pivotal-bot>Andrew Woods added comment: "How large have you tested this?" https://www.pivotaltracker.com/story/show/71236292
Esme Cowles added comment: "I've tested this up to 20GB (on a machine with 16GB RAM). I'm working on getting the tests to use my NFS st..." https://www.pivotaltracker.com/story/show/7123629209:33
* github-ff joins10:00
[fcrepo4] awoods closed pull request #349: Apply auto-hierarchy with the HierarchyConverter for objects, and fixed (master...feature/auto_hierarchy) http://git.io/aUC4Uw
* github-ff leaves
* escowles joins10:01
* github-ff joins10:02
[fcrepo4] escowles pushed 1 new commit to http-pid-minter: http://git.io/uHBCkw
fcrepo4/http-pid-minter 71196e3 Esmé Cowles: Using PoolingHttpClientConnectinoManager for pooled connections to remote PID minter
* github-ff leaves
* github-ff joins10:05
[fcrepo4] escowles pushed 1 new commit to large-file-range: http://git.io/MDsg1g
fcrepo4/large-file-range 351e851 Esmé Cowles: Adding author/since tags to class javadocs
* github-ff leaves
* longshou joins10:17
* travis-ci joins10:21
[travis-ci] futures/fcrepo4#1955 (http-pid-minter - 71196e3 : Esmé Cowles): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/e803d395d25c...71196e31dd44
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/25158596
* travis-ci leaves
<pivotal-bot>Esme Cowles added "Retroactively apply policy changes to existing repository content" https://www.pivotaltracker.com/story/show/7132141410:25
Esme Cowles added "Default policy on the root node" https://www.pivotaltracker.com/story/show/7132157010:26
* cjcolvar joins10:29
<pivotal-bot>Esme Cowles added "Internal SPARQL query should support specifying the subject URI instead of a variable" https://www.pivotaltracker.com/story/show/7132214010:32
* benpennell1 joins10:33
<tecoripa1>cbeer: I have a question about getting a list of objects from a properties Dataset omatching a given predicate10:42
cbeer: I'd like to run a method that looks something like this: Iterator<URIS> = find(DatasetGraph dg, Node predicate)10:43
cbeer: does that make sense? is there a utility somewhere in fcrepo that does something similar?10:44
cbeer: I've started down the path of doing that myself using the Jena find() methods in my class, but I thought if someone has already implemented a utility method that does that, or if there's a preferred way of getting that info in Fedora, I should follow that.10:46
<pivotal-bot>Kevin Clarke added "Turn on checkstyle rules for tests for jms indexer project" https://www.pivotaltracker.com/story/show/7132380410:51
Esme Cowles added "Support external datastreams" https://www.pivotaltracker.com/story/show/7132422210:56
<cbeer>tecoripa1: in what context?10:59
<benpennell1>mikeAtUVa: https://wiki.duraspace.org/display/FF/Versioning+Performance11:02
<mikeAtUVa>benpennell1: oooh.. pretty charts. Thanks!11:03
<tecoripa1>cbeer: retrieving a list of triple objects from node properties that are of given type, for use in XACML policies: goal is to allow users to create XACML policies that can filter, make decisions based on properties of nodes11:06
cbeer: example: users with a write role cannot delete nodes that have the property "notRemovable" set to true.11:10
<pivotal-bot>Benjamin Pennell started "Javadoc build warnings" https://www.pivotaltracker.com/story/show/69747108
Benjamin Pennell started "Create CND to express XACML policy structure" https://www.pivotaltracker.com/story/show/70689520
Gregory Jansen added "Add API support for root fedora object" https://www.pivotaltracker.com/story/show/7132661811:12
* dwilcox leaves11:36
<pivotal-bot>Andrew Woods edited "Allow for practical use of read-only filesystem federation." https://www.pivotaltracker.com/story/show/71234576
<tecoripa1>cbeer: here's what I have:11:37
cbeer: properties = resource.getPropertiesDataset(idTranslator).asDatasetGraph();
cbeer: final Iterator<Quad> quad =11:38
properties.find(Node.ANY, Node.ANY, NodeFactory.createURI(attributeType.toString()), Node.ANY);
<pivotal-bot>Gregory Jansen edited "Add API support for root fedora object" https://www.pivotaltracker.com/story/show/7132661811:41
<awoods>longshou: ready?11:44
longshou: https://plus.google.com/hangouts/_/event/c1glu6soq43r1rr6ou17qtobug811:45
* dwilcox joins11:47
* dwilcox leaves
<benpennell1>mikeAtUVa: this may be some additional jcr leakage. I can visit jcr:content on a datastream (instead of fcr:content) to get a description of the jcr:content node, although the path shown in the page claims it is fcr:content.11:49
And even weirder, if i use curl to create a new version of jcr:content/fcr:versions it works, but fcr:content/fcr:versions throws an error "Error converting "fcr:content" from String to a Name". I can also get to jcr:content/fcr:versions in the html interface but not fcr:content11:50
fcr:content/fcr:versions that is
<mikeAtUVa>benpennell1: just so I'm clear... you can GET "http://localhost:8080/object/jcr:content" and it behaves just like "http://localhost:8080/object/fcr:content"..11:51
<benpennell1>yes, if "object" is the path to a datastream11:52
<mikeAtUVa>benpennell1: also you can't post to "http://localhost:8080/object/fcr:content/fcr:versions" to create a new version?
benpennell1: again, with the caveat about the example path being a datastream...
curl -X POST http://localhost:8080/rest/05/32/c8/bc/0532c8bc-ea0d-43d0-9866-ac0ce492d54b/06/59/aa/51/0659aa51-cb59-4c26-96d7-1370dca418da/fcr:content/fcr:versions
Error converting "fcr:content" from String to a Name
no error if it is jcr:content instead, and a new version shows up if you got to its versions page11:53
<mikeAtUVa>benpennell1: But if you POST to create a new version at the datastream path, it creates a new version of the content?
<benpennell1>no, that just makes a new version of the datastream (haven't tried it with autoversioning turned on though)11:54
<pivotal-bot>Esme Cowles added comment: "This PR from yulgit1 addresses this, but has issues (and doesn't build now): https://github.com/futures/fcre..." https://www.pivotaltracker.com/story/show/68016728
<benpennell1>it actually makes two new versions of the datastream show up, but that's apparently a bug longshou is already working on
<mikeAtUVa>the direct access to jcr:content doesn't surprise me... and as long as we don't link to it anywhere it's probably not a huge concern, though I would like to fix that as part of this effort.
benpennell1: I think it makes a version of the datastream and a version of the content node... and the bug he is working on is that it presents both of them in the RDF for the datastream's fcr:versions endpoint.11:55
benpennell1: I'm not sure that we intend for people to interact with fcr:content as if it were a full-fledged resource but instead it's a special part of a datastream that's accessible through fcr:content. I wonder if /fcr:content/fcr:export work? Should it?11:57
<benpennell1>ah, that might make sense with some of the weirdness i was seeing, the whole reason I was making calls directly to jcr:content was because I was trying to figure out how you were supposed to get a new version of a datastream when you updated its content (without using autoversioning)11:58
for what its worth, /fcr:content/fcr:export on a datastream throws a 404 for me11:59
/jcr:content/fcr:export works though
as to whether or not you'd want it to work, i'm not sure why you'd want to address it separately from its datastream12:01
* dwilcox joins12:06
* github-ff joins12:15
[fcrepo4] awoods closed pull request #270: refactored accessroles properties to exist on the node they are relative... (master...rootread2) http://git.io/Hma5cw
* github-ff leaves
* mjgiarlo joins12:16
escowles: can you (or someone else) give a quick rundown of how projection's black magic works? so I configure F4 to look at a directory on my filesystem. does F4 then mint IDs for every subdir and file under that dir? is that how projected content is modeled internally?12:17
I forwarded the recent Fedora 4 blast to other folks in my department and now they're all excited and want to know how it works. ;)12:18
(at a moderately high level -- I can't just point them at modeshape's tech docs for the filesystem connector.)12:19
<escowles>mjgiarlo: yes, it does mint UUIDs for everything and model them in the repo, but you don't need to know about that when you're using it -- you just map a filesystem path to a REST API path, and then work with the REST API path
* edInCo joins12:20
<cbeer>oh, right. but the UUIDs is essentially an implementation detail to play nice with JCR
* benpennell1 leaves12:21
<escowles>cbeer: right -- and mikeAtUVa is working on hiding UUIDs and similar JCR stuff so they won't show up in the RDF or HTML UI
<mjgiarlo>so projection almost has a sufia-like assumption that files are first-class objects
<cbeer>escowles: hm. i'm not sure i like the sound of that. it's nice to have a non-path-based identifier sometimes.
(e.g. i know i was looking for [abc-123-qwerty-123456]... i don't care where it is in the hierarchy, just give me the object)12:22
<escowles>cbeer: but we don't have that in the REST API right now -- you can't do anything in the REST API using a node's UUID12:23
<cbeer>escowles: what? you used to be able to
<escowles>cbeer: let me look at the REST API docs a minute...12:24
<cbeer>hm. it used to be something like:12:25
i wonder if that got dropped in ajs6f's identifier refactor :/
oh, no12:26
it'd help if i used the right escape sequence
* cbeer
escowles: ^12:27
<mjgiarlo>escowles++ cbeer++ # thx. I think I know enough now to explain it without confusing people. :)12:29
<escowles>cbeer: ok, that does work -- is that because session.getNode() handles that?12:30
<cbeer>escowles: yep.
<escowles>cbeer: it would be nice if that was documented somewhere, if it's something we are going to support...12:31
<pivotal-bot>Longshou Situ added comment: "PR updated: https://github.com/futures/fcrepo4/pull/352" https://www.pivotaltracker.com/story/show/7114057012:32
* benpennell1 joins
<escowles>cbeer: awoods is worried that we don't have any control over the UUID generation, they may change if content is moved/migrated/etc. and we wouldn't be able to do anything about it12:44
and if that's the case, do we want to expose the UUID identifers and functionality, and run the risk they will break in the future in certain circumstances?12:45
<cbeer>escowles: as long as we make it clear it's an implementation detail and an administrative tool, i don't see the problem.12:46
no one would ever take that UUID and build a whole infrastructure around it!
<escowles>mjgiarlo: before you run away screaming, let me know if https://wiki.duraspace.org/display/FF/Federation is good enough for your purposes, or if there's something we should add or expand12:47
cbeer: you say that -- but i've seen people use ephemeral identifiers before...12:48
<cbeer>but, hey, they're unique.
they just might not be stable.
<mjgiarlo>escowles: for *me*, yes. for the folks who are asking me questions, no. this was helpful, though, so thanks :)12:49
<escowles>mjgiarlo: patches welcome, as always... if you find yourself writing an email or something, anyway...12:50
<mjgiarlo>escowles: I'll gist what I wrote them, and maybe you can tell me how accurate it was. ;)
escowles: https://gist.github.com/mjgiarlo/a21311dd898e9e365a1712:56
(not sure why the bullets aren't formatting properly)12:57
<escowles>yep, pretty much -- the only thing i see is that files don't have versions (stuff blows up if you try to do versioning with filesystem)12:59
<mjgiarlo>OK, great13:03
I suppose F4 *could* do some semblance of versioning13:04
but I'll clarify that that isn't currently supported
<escowles>we've talked about that -- we could have an implementation that had disk-based versioning, etc. -- but that would be a new implementation that extended the current one13:05
* mjgiarlo nods
things are so easy when they are conceptual...13:06
* github-ff joins13:07
[fcrepo4] awoods pushed 1 new commit to hierarchy: http://git.io/rGfp4w
fcrepo4/hierarchy 3b57136 lsitu: Fix the HierarchyConverter to produce a consistent result...
* github-ff leaves
<pivotal-bot>Andrew Woods added comment: "Resolved by: https://github.com/futures/fcrepo4/commit/3b57136a35a45deaaa37d3a367718d5a1ce356e0" https://www.pivotaltracker.com/story/show/71140570
Andrew Woods delivered "Hierarchy converter implementation" https://www.pivotaltracker.com/story/show/7114057013:08
* github-ff joins
[fcrepo4] awoods closed pull request #352: Fixed the HierarchyConverter to produce a consistency result; (hierarchy...hierarchy) http://git.io/FOhtqQ
* github-ff leaves
<mjgiarlo>escowles: engineers are always sullying my pure, perfect concepts.13:09
<escowles>mjgiarlo: you have to admit it's fair -- architects are always interrupting my pure, perfect hacking sessions... ;-P13:10
* travis-ci joins13:25
[travis-ci] futures/fcrepo4#1959 (hierarchy - 3b57136 : lsitu): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/3b2a723b2302...3b57136a35a4
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/25173883
* travis-ci leaves
* github-ff joins
[fcrepo-module-auth-xacml] gregjan opened pull request #11: latest installment of xacml goodness (master...master) http://git.io/f-8Zcg
* github-ff leaves
* mjgiarlo leaves13:26
* cjcolvar leaves13:34
* gregjansen leaves13:36
* benpennell1 leaves14:09
* benpennell1 joins14:47
* github-ff joins14:59
[fcrepo4] ksclarke opened pull request #357: Removed requirement for fedoraUser in auth'ed connections (master...remove-fedoraUser-dependency-70306268) http://git.io/NNA0Hw
* github-ff leaves
<pivotal-bot>Kevin Clarke added comment: "https://github.com/futures/fcrepo4/pull/357" https://www.pivotaltracker.com/story/show/70306268
Kevin Clarke started "Refactor fedoraUser AuthZ expectation" https://www.pivotaltracker.com/story/show/7030626815:00
Kevin Clarke finished "Refactor fedoraUser AuthZ expectation" https://www.pivotaltracker.com/story/show/70306268
Kevin Clarke started "Create Maven BOMs for fcrepo dependencies" https://www.pivotaltracker.com/story/show/6357145615:02
Andrew Woods added "Transparent filesystem persistence" https://www.pivotaltracker.com/story/show/7134708615:05
* gregjansen joins15:17
<pivotal-bot>Gregory Jansen started "Integration tests for basic role XACML policies" https://www.pivotaltracker.com/story/show/70771806
* github-ff joins
[fcrepo-module-auth-xacml] sprater pushed 2 new commits to master: http://git.io/5MyB6w
fcrepo-module-auth-xacml/master c1292fa Andrew Woods: Add basic TripleAttributeFinderModule unit tests...
fcrepo-module-auth-xacml/master d03429c sprater: Merge pull request #9 from awoods/triple-finder...
* github-ff leaves
* 18VAAU2WS joins
[fcrepo-module-auth-xacml] sprater closed pull request #9: Add basic unit tests for TripleAttributeFinderModule (master...triple-finder) http://git.io/b_JWaw
* 18VAAU2WS leaves
<pivotal-bot>Gregory Jansen delivered "XACML EnvironmentAttributeFinderModule" https://www.pivotaltracker.com/story/show/7069601815:18
Gregory Jansen added comment: "This is all in master now" https://www.pivotaltracker.com/story/show/70943414
* github-ff joins
[fcrepo-module-auth-xacml] sprater pushed 3 new commits to master: http://git.io/Yh6YzQ
fcrepo-module-auth-xacml/master d65db28 Gregory Jansen: added test for building eval context...
fcrepo-module-auth-xacml/master bef2561 Gregory Jansen: renamed initial policies to match their IDs...
fcrepo-module-auth-xacml/master 4ee4b22 sprater: Merge pull request #11 from gregjan/master...
* github-ff leaves
* 18VAAU2XH joins
[fcrepo-module-auth-xacml] sprater closed pull request #11: latest installment of xacml goodness (master...master) http://git.io/f-8Zcg
* 18VAAU2XH leaves
<pivotal-bot>Gregory Jansen finished "Create XACML evaluation context in authz delegate" https://www.pivotaltracker.com/story/show/70943414
Benjamin Pennell finished "Create CND to express XACML policy structure" https://www.pivotaltracker.com/story/show/7068952015:19
* github-ff joins15:20
[fcrepo4] escowles pushed 1 new commit to http-pid-minter: http://git.io/CJsYxw
fcrepo4/http-pid-minter 48327b9 Esmé Cowles: Making master.xml spring config file to bundle config files and avoid wildcard include, breaking out minter config and making it configurable with system property fcrepo.minter.config
* github-ff leaves
<pivotal-bot>Gregory Jansen added comment: "Ben, please add a link in the wiki to the cnd file in git repository." https://www.pivotaltracker.com/story/show/70689520
* travis-ci joins15:26
[travis-ci] futures/fcrepo-module-auth-xacml#29 (master - d03429c : sprater): The build was broken.
[travis-ci] Change view : https://github.com/futures/fcrepo-module-auth-xacml/compare/d1e50df8d3fb...d03429c3ad8d
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo-module-auth-xacml/builds/25183194
* travis-ci leaves
* travis-ci joins15:29
[travis-ci] futures/fcrepo-module-auth-xacml#30 (master - 4ee4b22 : sprater): The build was broken.
[travis-ci] Change view : https://github.com/futures/fcrepo-module-auth-xacml/compare/d03429c3ad8d...4ee4b22e7f4c
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo-module-auth-xacml/builds/25183233
* travis-ci leaves
* travis-ci joins15:34
[travis-ci] futures/fcrepo4#1961 (hierarchy - 917851c : Andrew Woods): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/3b57136a35a4...917851c1d67d
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/25183232
* travis-ci leaves
* travis-ci joins15:36
[travis-ci] futures/fcrepo4#1962 (http-pid-minter - 48327b9 : Esmé Cowles): The build passed.
[travis-ci] Change view : https://github.com/futures/fcrepo4/compare/71196e31dd44...48327b97da54
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo4/builds/25183341
* travis-ci leaves
<pivotal-bot>Esme Cowles added comment: "I've added a feature and configuration pages to the wiki for the HttpPidMinter: ""15:49
https://wiki.duraspace.org/..." https://www.pivotaltracker.com/story/show/70678050
* dwilcox leaves15:52
<pivotal-bot>Benjamin Pennell added comment: "added it" https://www.pivotaltracker.com/story/show/7068952016:02
* github-ff joins16:11
[fcrepo-module-auth-xacml] sprater opened pull request #12: Clean up XACML policies (master...triplefinder) http://git.io/FrwcaA
* github-ff leaves
<pivotal-bot>Gregory Jansen delivered "Create CND to express XACML policy structure" https://www.pivotaltracker.com/story/show/7068952016:13
* dwilcox joins
<pivotal-bot>Mike Durbin added comment: "While I think it would be nice to hide the mix:versionable mixin, I'm concerned that as long as we allow peo..." https://www.pivotaltracker.com/story/show/7126623216:15
* github-ff joins16:18
[fcrepo-module-auth-xacml] gregjan pushed 11 new commits to master: http://git.io/E4sfkA
fcrepo-module-auth-xacml/master 018fdea sprater: Merge remote-tracking branch 'awoods/triple-finder' into triplefinder
fcrepo-module-auth-xacml/master 796d181 sprater: XACML finder for triple attributes
fcrepo-module-auth-xacml/master c470a74 sprater: XACML finder module for triples
* github-ff leaves
* travis-ci joins16:32
[travis-ci] futures/fcrepo-module-auth-xacml#32 (master - b84b025 : Gregory Jansen): The build has errored.
[travis-ci] Change view : https://github.com/futures/fcrepo-module-auth-xacml/compare/4ee4b22e7f4c...b84b025454b0
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo-module-auth-xacml/builds/25187946
* travis-ci leaves
* github-ff joins16:33
[fcrepo-module-auth-xacml] sprater force-pushed master from b84b025 to 3cf3a7c: http://git.io/fUNyNw
fcrepo-module-auth-xacml/master 3cf3a7c sprater: Fix XACML policy XML validation errors
* github-ff leaves
* travis-ci joins16:44
[travis-ci] futures/fcrepo-module-auth-xacml#33 (master - 3cf3a7c : sprater): The build failed.
[travis-ci] Change view : https://github.com/futures/fcrepo-module-auth-xacml/compare/b84b025454b0...3cf3a7ce1731
[travis-ci] Build details : http://travis-ci.org/futures/fcrepo-module-auth-xacml/builds/25189198
* travis-ci leaves
<pivotal-bot>Mike Durbin started "Omit JCR locking triples from object RDF." https://www.pivotaltracker.com/story/show/7126628016:46
* dwilcox leaves16:47
<ksclarke>might be of interest to folks (sent to me today by a coworker): a chrome extension that displays github repos in tree form on the left side of your screen -- https://chrome.google.com/webstore/detail/octotree/bkhaagjahfmjljalopjnoealnfndnagc/16:52
<pivotal-bot>Mike Durbin edited "Omit protected triples from object RDF." https://www.pivotaltracker.com/story/show/7126623216:57
Mike Durbin deleted "Omit JCR locking triples from object RDF." https://www.pivotaltracker.com/story/show/71266280
* github-ff joins17:15
[fcrepo-module-auth-xacml] sprater opened pull request #13: Check for nulls in FedoraEvaluationCtxBuilder (master...evalctx-nullfix) http://git.io/lqlwlw
* github-ff leaves
* github-ff joins17:16
[fcrepo4] awoods pushed 1 new commit to hierarchy: http://git.io/AAX1Yw
fcrepo4/hierarchy 9b1ff36 lsitu: Refactor CRUD operations in FedoraNodes and FedoraContent for transparent auto-hierarchy support....
* github-ff leaves
<pivotal-bot>Andrew Woods added comment: "Partially addressed by: ""17:17
..." https://www.pivotaltracker.com/story/show/71230608
Andrew Woods edited "Refactor CRUD in FedoraNodes and FedoraContent for transparent auto-hierarchy support." https://www.pivotaltracker.com/story/show/71230608
Andrew Woods rejected "Refactor CRUD in FedoraNodes and FedoraContent for transparent auto-hierarchy support." https://www.pivotaltracker.com/story/show/71230608
Andrew Woods rejected "Hierarchy: 404 on datastream creation" https://www.pivotaltracker.com/story/show/7098515017:22
Andrew Woods rejected "Hierarchy: 404 on object PUT" https://www.pivotaltracker.com/story/show/70984768
Andrew Woods rejected "Hierarchy: Datastreams created at a level too high" https://www.pivotaltracker.com/story/show/70984934
Andrew Woods rejected "Hierarchy: Location response URL on object creation is unexpected" https://www.pivotaltracker.com/story/show/70984436
<awoods>longshou: ping17:23
<awoods>longshou: I pushed your PR: https://github.com/futures/fcrepo4/pull/35417:24
longshou: along with some code-review updates: https://github.com/awoods/fcrepo4/commit/87e0d5f3bb4086acb3e2fe633c7f0bf5d41e47cf
<longshou>Really? I am trying to merge the hierarchyCRUD branch but got error after reinitiate the hierarchy branchy.17:25
<awoods>longshou: You have new commits?
longshou: as it stands right now, your PRs have all been committed to "hierarchy"17:26
longshou: there are a few issues, however...
<longshou>What issues?17:27
<awoods>longshou: we need to bring back to life the @Ignored tests
longshou: and the HTML UI does not work
longshou: When I create an object via curl, then go to the UI... boom!
longshou: do you have new commits? or have you been working on other tickets?17:28
<longshou>I don’t see it in my local test. Is it related to some latest updates?
<awoods>longshou: you do not see what?
<longshou>Yes, the html view for fcr:versions.17:29
<awoods>longshou: try this:
<longshou>The error 404.
<awoods>mvn jetty:run -pl fcrepo-webapp17:30
<longshou>Have you ever try the REST API?
<awoods>curl -v -XPUT http://localhost:8080/rest/book
longshou: then go to the HTML UI and you will not see "book", but instead you will see a path segment17:31
longshou: then click on that path segment... boom!
<longshou>Okay, I will check it out to take a look to see what wrong.
awoods: ping17:35
* github-ff joins
[fcrepo-jms-indexer-pluggable] ksclarke opened pull request #40: Turned on checkstyle checks for tests and added @author tag where missing (master...add-tests-to-checkstyle-71323804) http://git.io/R1IRsg
* github-ff leaves
<longshou>You lost the last commit on https://github.com/futures/fcrepo4/pull/354
awoods: https://github.com/lsitu/fcrepo4/commit/9b5859978faecaa4389c8e4cdf9dec4413e0643317:37
<awoods>longshou: no, I merged it with https://github.com/awoods/fcrepo4/commit/87e0d5f3bb4086acb3e2fe633c7f0bf5d41e47cf
longshou: see: https://github.com/futures/fcrepo4/commit/9b1ff36ebe80a8464c1ddf233ce45b139123bcb717:38
<longshou>Now I got it. Thanks. I am checking it out now…17:39
<pivotal-bot>Kevin Clarke added comment: "https://github.com/futures/fcrepo-jms-indexer-pluggable/pull/40" https://www.pivotaltracker.com/story/show/7132380417:43
Kevin Clarke added comment: "Might be nice to do the checkstyle updates before we do this." https://www.pivotaltracker.com/story/show/7071858217:53
<cbeer>escowles: ping?18:34
* github-ff joins18:35
[fcrepo4] lsitu opened pull request #358: Feature/versions html (master...feature/versions_html) http://git.io/AayCQw
* github-ff leaves
<pivotal-bot>Longshou Situ added comment: "https://github.com/fcrepo4/fcrepo4/pull/358" https://www.pivotaltracker.com/story/show/70910668
Longshou Situ finished "Clean-up HTML view of fcr:versions" https://www.pivotaltracker.com/story/show/70910668
Longshou Situ added comment: "@awoods The html view for fcr:versions should be improved now: https://github.com/fcrepo4/fcrepo4/pull/358" https://www.pivotaltracker.com/story/show/7047505818:38
Longshou Situ finished "Improve and simplify fcr:versions response triples." https://www.pivotaltracker.com/story/show/70475058
<awoods>cbeer: escowles is cooking
<cbeer>awoods: figures.18:39
<escowles>cbeer: cooking shepherd's pie & drinking beer...18:47
<cbeer>RDF is hard.18:48
<longshou>awoods: ping19:07
<longshou>The problem is in -v option in “curl -v -XPUT http://localhost:8080/rest/book
It works when I tried “curl -X PUT http://localhost:8080/rest/book”. Version is another issue we need to fix next. Ideas?19:09
<awoods>longshou: what?? you are saying "curl -v" is problematic?
longshou: "-v" is verbose
longshou: I need to step away for dinner...19:10
longshou: escowles demands it
<longshou>Sure. But -v option touches “jcr:system/jcr:versionStorage/”19:11
<awoods>longshou: I am not sure that makes sense to me... it is hard to imagine "curl -v" has anything to do with fcrepo versioning... but I will take a look later.19:13
* benpennell1 leaves19:46
* edInCo leaves19:49
* escowles leaves20:46
* github-ff joins21:55
[fcrepo4] mikedurbin opened pull request #359: Treat protected properties as "internal" (master...jcr-leak) http://git.io/OOEQog
* github-ff leaves
* gregjansen leaves21:57
* gregjansen joins22:03
* gregjansen leaves22:19
* gregjansen joins
* gregjansen leaves22:47
* md5wz__ joins22:56
* mikeAtUVa leaves22:59
* tecoripa1 leaves23:11
* gregjansen joins23:15
* benpennell1 joins23:17
* benpennell1 leaves
* gregjansen leaves23:19
* gregjansen joins23:50
* ksclarke leaves00:00
* gregjansen leaves00:37

Generated by Sualtam