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

Mode 1587 - Updated the ModeShape kit so that it works with AS 7.2.0 #724

Merged
merged 3 commits into from
Mar 4, 2013

Conversation

hchiorean
Copy link
Member

For the kit/integration tests to be able to build & run, one needs do a mvn install -Prelease locally, on an AS 7.2.0.Final project.

For users to still be able to build ModeShape without problems, the AS7 kit modules have been moved from the standard build list and into dedicated profiles: integration, assembly, performance.

This PR contains an additional commit, which cleans up/changes IPv4 and IPv6 system properties usage, as it turns out they can have a significant performance impact.

Horia Chiorean added 3 commits March 4, 2013 14:47
…S7.2 server and re-enabled the integration tests modules. However, for a local build to work, one needs for the time being a locally installed (via Maven) version of AS7.2.

 In the process of updating the kit, a couple of other things were updated:
 - exception processing around WritableSessionCache#persistChanges
 - logging of exceptions / errors in the ModeShape Webdav Servlet
…v6 profiles. By default, neither IPv4 nor IPv6 will be forced, meaning that whatever the underlying OS supports will be used (see http://docs.oracle.com/javase/1.5.0/docs/guide/net/ipv6_guide/index.html for more information)

Also, It seems that on Windows (at least) JGroups 3.2 is a lot faster when using IPv6 than it is when using IPv4, so a new OS-dependent profile was added to modeshape-jcr. Also, it seems Arquillian doesn't work with IPv6, so IPv4 was enforced for the integration tests.
…le into dedicated profiles, so that user can still run "mvn clean install". We should keep this at least until we know what the Maven availability of the AS7 dist zip will be.
@rhauch rhauch merged commit 41d2708 into ModeShape:master Mar 4, 2013
@rhauch
Copy link
Contributor

rhauch commented Mar 4, 2013

Merged into the 'master' branch.

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.

2 participants