[jira] [Commented] (DOXIA-543) maven-doxia-markdown fails in parallel builds

["Hudson (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/DOXIA-543?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15852995#comment-15852995 ] 

Hudson commented on DOXIA-543:
------------------------------

SUCCESS: Integrated in Jenkins build doxia-all #325 (See [https://builds.apache.org/job/doxia-all/325/])
[DOXIA-543] maven-doxia-markdown fails in parallel builds (michaelo: [http://svn.apache.org/viewvc/?view=rev&rev=1781714])
* (edit) ./doxia/doxia-modules/doxia-module-markdown/src/main/java/org/apache/maven/doxia/module/markdown/MarkdownParser.java


> maven-doxia-markdown fails in parallel builds
> ---------------------------------------------
>
>                 Key: DOXIA-543
>                 URL: https://issues.apache.org/jira/browse/DOXIA-543
>             Project: Maven Doxia
>          Issue Type: Bug
>          Components: Module - Markdown
>    Affects Versions: 1.7
>            Reporter: Daniel Wegener
>            Assignee: Michael Osipov
>             Fix For: 1.8
>
>   Original Estimate: 10m
>  Remaining Estimate: 10m
>
> The PegDownProcessor used at https://github.com/apache/maven-doxia/blob/doxia-1.7/doxia-modules/doxia-module-markdown/src/main/java/org/apache/maven/doxia/module/markdown/MarkdownParser.java#L72 is shared between multiple concurrent executions of the maven-site-plugin.
> This leads to random errors (NPEs, ClassCastException) because the parser state machine is reused by multiple threads.
> It seem not to be enough to make the field non-static since the whole MarkdownParser instance seem to be shared between different threads.
> The PegDown Javadoc explicitly states that a PegDownProcessor is not thread-safe: https://github.com/sirthias/pegdown/blob/master/src/main/java/org/pegdown/PegDownProcessor.java
> Possible solutions:
> - Put the PegDownProcessor into a threadLocal.
> - Instanciate the PegDownProcessor in the toHtml method.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)