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

Reimplement WAR unarchiver to use plexus-archiver 2.3 for major SDK performance improvements #124

Closed
mindthegab opened this Issue Jul 20, 2014 · 3 comments

Comments

Projects
None yet
1 participant
@mindthegab
Contributor

mindthegab commented Jul 20, 2014

From gabriele...@alfresco.com on June 12, 2013 11:55:06

WAR unpacking is unnecessarily slow because chmod is called on every uncompressed file, forking an OS process (instead of using java NIO native capabilities).

Using plexus-archiver 2.3 actually fixes the issue but currently the alfresco-maven-plugin is not compatible with it.

Simply changing the dependency and implementing missing methods should do it, and give the whole SDK a huge performance boost.

Original issue: http://code.google.com/p/maven-alfresco-archetypes/issues/detail?id=124

@mindthegab

This comment has been minimized.

Contributor

mindthegab commented Jul 20, 2014

From colum...@gmail.com on June 21, 2013 04:16:02

Status: Started
Owner: colum...@gmail.com
Labels: -Priority-Medium Priority-Critical Milestone-Release-sdk-1.0.2

@mindthegab

This comment has been minimized.

Contributor

mindthegab commented Jul 20, 2014

From colum...@gmail.com on July 02, 2013 03:25:47

This is fixed in trunk, needs to be verified.

In order to verify, please the maven-war-plugin using an AMP in the overlay (e.g. allinone)

Status: Fixed

@mindthegab

This comment has been minimized.

Contributor

mindthegab commented Jul 20, 2014

From colum...@gmail.com on July 02, 2013 08:25:35

Verified in all-in-one archetype.

Strong (about 10x) performance improvement, on a mvn clean install:

alfresco.war --> 12s
share.war --> 14s

Status: Verified

@mindthegab mindthegab closed this Jul 20, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment