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

ASF GitHub Bot commented on MNG-7476:
-------------------------------------

gnodet commented on code in PR #736:
URL: https://github.com/apache/maven/pull/736#discussion_r874048159


##########
maven-core/src/main/java/org/apache/maven/lifecycle/internal/MojoExecutor.java:
##########
@@ -282,6 +308,32 @@ private Lock getProjectLock( MavenSession session )
         }
     }
 
+    static class OwnerReentrantLock extends ReentrantLock
+    {
+        @Override
+        public Thread getOwner()
+        {
+            return super.getOwner();
+        }
+    }
+
+    static class OwnerReentrantReadWriteLock extends ReentrantReadWriteLock
+    {
+        @Override
+        public Thread getOwner()
+        {
+            return super.getOwner();
+        }
+    }
+
+    private static void warn( String msg )
+    {
+        for ( String s : MessageHelper.formatWarning( msg ) )
+        {
+            LOGGER.warn( s );
+        }

Review Comment:
   Yes, but I don't think slf4j provide any api to _lock_ the appender.  In 
addition, any build-in new line will break the formatting box.  But in a 
concurrent build, you definitely want to use _mvnd_ which brings back things in 
order :-)





> Display a warning when an aggregator mojo is locking other mojo executions
> --------------------------------------------------------------------------
>
>                 Key: MNG-7476
>                 URL: https://issues.apache.org/jira/browse/MNG-7476
>             Project: Maven
>          Issue Type: Improvement
>    Affects Versions: 3.8.5
>            Reporter: Guillaume Nodet
>            Assignee: Guillaume Nodet
>            Priority: Major
>             Fix For: 3.8.6, 3.9.0, 4.0.0-alpha-1, 4.0.0
>
>




--
This message was sent by Atlassian Jira
(v8.20.7#820007)

Reply via email to