[
https://jira.codehaus.org/browse/MINDEXER-50?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=292787#comment-292787
]
Olivier Lamy commented on MINDEXER-50:
--------------------------------------
I frankly don't understand the pattern.
if (log.isDebugEnabled() )
{
log.info("blabla);
}
Looks weird for me. should be log.isInfoEnabled or log.debug
> Logging of parsing exceptions should be handled consistently in
> org.apache.maven.index.creator.MavenPluginArtifactInfoIndexCreator
> ----------------------------------------------------------------------------------------------------------------------------------
>
> Key: MINDEXER-50
> URL: https://jira.codehaus.org/browse/MINDEXER-50
> Project: Maven Indexer
> Issue Type: Bug
> Affects Versions: 4.1.2
> Reporter: Peter Lynch
>
> This patch makes the logging of artifact parsing exceptions consistent
> between org.apache.maven.index.creator.MavenPluginArtifactInfoIndexCreator
> and org.apache.maven.index.creator.MavenArchetypeArtifactInfoIndexCreator
> https://github.com/apache/maven-indexer/pull/2.diff
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://jira.codehaus.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
