Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update ModeShape -> 4.4.0, and ISPN -> 7.2.3 #897

Closed
wants to merge 3 commits into from
Closed

Conversation

awoods
Copy link

@awoods awoods commented Sep 1, 2015

* See the License for the specific language governing permissions and
* limitations under the License.
*/
package org.fcrepo.kernel.impl.utils.impl;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wasn't this file removed a while ago? Certainly the package name org.fcrepo.kernel.impl shouldn't be there.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Re-removed.

<persistence passivation="false">
<file-store shared="false"
preload="false"
fetch-state="false"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a reason this is different from fetchPersistentState in the old config? Reading the user guide explanation of the fetch-state option, it sounds like we want fetch-state=true: http://infinispan.org/docs/7.2.x/user_guide/user_guide.html#_configuration_2

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @escowles. Nice catch.

- Change fetch-state to 'true'
<transaction transaction-manager-lookup="org.infinispan.transaction.lookup.GenericTransactionManagerLookup" mode="NON_XA" locking="PESSIMISTIC"/>
<eviction max-entries="500" strategy="LIRS" thread-policy="DEFAULT"/>
<persistence>
<store:leveldb-store passivation="false" path="${fcrepo.ispn.repo.cache:target/repo}/data">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried running the modeshape clustered config (config/clustered/repository.json) and got an error unless I removed the passivation attribute from this line and line 27.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Out of curiosity, did you run it in a cluster or as a single instance?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried to run a second node on the same machine, but it didn't work. I think it's worth testing clustering on two nodes before upgrading, so I can do that shortly and see if I can get it to work.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As a side note, I also did some testing with running/populating a 4.3.0 version of Fedora, then starting up a Fedora with these upgraded configs on the same fcrepo.home... but more testing around backward/forward compatibility would be useful.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've tried running the clustered config on two machines, and I am getting an stacktrace and total failure on both machines. Here's a gist of my script (based on the system properties specified at https://wiki.duraspace.org/display/FEDORA4x/Deploying+a+Fedora+Cluster) and the error I'm getting:

https://gist.github.com/escowles/d979a15fd8ec5b7c99f3

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is that with passivation= true or false?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's with the passivation attribute removed -- I get an error if it is present at all.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Error:
org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'modeshapeRepofactory': Invocation of init method failed; nested exception is org.fcrepo.kernel.api.exception.RepositoryRuntimeException: org.infinispan.commons.CacheConfigurationException: javax.xml.stream.XMLStreamException: ParseError at [row,col]:[14,100]|Message: Unexpected attribute 'passivation' encountered

@awoods
Copy link
Author

awoods commented Sep 17, 2015

This work is proceeding on the following branch ('modeshape-4.4.0'):
https://github.com/fcrepo4/fcrepo4/tree/modeshape-4.4.0

...and associated PR:
#910

@awoods awoods closed this Sep 17, 2015
@awoods awoods deleted the fcrepo-1581 branch September 17, 2015 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants