Implementation for no binary content jcr/xml file system persistence. #47
Conversation
@@ -38,11 +38,17 @@ | |||
</bean> | |||
<!-- Solr Indexer END--> | |||
|
|||
<!-- jcr/xml persistence Indexer --> | |||
<bean id="jcrXmlPersist" class="org.fcrepo.indexer.persistence.JcrXmlPersistenceIndexer"> | |||
<property name="path" value="${jcr.persist.dir:./target/test-classes/jcrXml/}"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Create a default value that is more suitable for production use (${java.io.tmpdir}?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about ${java.io.dir}? The persisted jcr/xml looks more like permanent storage compared to "tmp".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My point is that we want a reasonable default value, and "./target/test-classes/jcrXml" does not seem like a reasonable default value.
Maybe: "jcrXml-storage" or the java tmp dir.
configuring file path as suggested.
Okay, see lsitu@bb1f5eb. |
* @author ajs6f | ||
* @date Aug 19, 2013 | ||
*/ | ||
public class JcrXmlPesistenceIndexerTest { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix typo:
JcrXmlPesistenceIndexerTest should be "JcrXmlPersistenceIndexerTest"
system persistence.
Corrected at commit lsitu@e8894a6. |
Resolved with: b78416a Addresses: https://www.pivotaltracker.com/story/show/74702010 |
https://www.pivotaltracker.com/s/projects/684825/stories/74702010
We should be able to persist jcr/xml with no binary contents to the file system now. The location can be configured with parameter jcr.persist.dir as defined in in fcrepo-jms-indexer-webapp/src/main/resources/spring/indexer-core.xml, and the filename for the jcr/xml will be {id}-jcr.xml.