[ 
https://jira.codehaus.org/browse/MCHECKSTYLE-207?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Dennis Lundberg closed MCHECKSTYLE-207.
---------------------------------------

       Resolution: Fixed
    Fix Version/s: 2.12
         Assignee: Dennis Lundberg

Patch applied in 
[r1573348|http://svn.apache.org/viewvc?view=revision&revision=1573348]. Thank 
you!

> file handle leak - leading to failed builds
> -------------------------------------------
>
>                 Key: MCHECKSTYLE-207
>                 URL: https://jira.codehaus.org/browse/MCHECKSTYLE-207
>             Project: Maven Checkstyle Plugin
>          Issue Type: Bug
>         Environment: Linux
>            Reporter: James Nord
>            Assignee: Dennis Lundberg
>             Fix For: 2.12
>
>         Attachments: MCHECKSTYLE-207_PATCH_V1 (1).txt
>
>
> The plugin sets the classpath to load as a URLClassloader populated with all 
> the URLs to the projects dependencies (jars) and output folders.
> However the URLClassloader is never closed (only available in JDK1.7+) and 
> for a large multi-module project with a large number of dependencies this 
> leak can seriously add up (which if gc doesn't kick in at the right moment) 
> will lead to random failures due to java.io.IOException: error=24, Too many 
> open files.
> see 
> http://svn.apache.org/viewvc/maven/plugins/tags/maven-checkstyle-plugin-2.11/src/main/java/org/apache/maven/plugin/checkstyle/DefaultCheckstyleExecutor.java?revision=1540890&view=markup
> Line 162 where the URLClassloader is created - but never cleaned up.
> The fix is relatively trivial if it is only to be fixed for 1.7+ JVMs (and 
> use reflection if need to be compatable with < 1.7).
> Otherwise a custom classloader will be needed.



--
This message was sent by Atlassian JIRA
(v6.1.6#6162)

Reply via email to