joseluisll commented on code in PR #8682:
URL: https://github.com/apache/hadoop/pull/8682#discussion_r3813797753


##########
hadoop-common-project/hadoop-common/pom.xml:
##########
@@ -541,12 +541,6 @@
           <systemPropertyVariables>
             <runningWithNative>${runningWithNative}</runningWithNative>
           </systemPropertyVariables>
-          <properties>
-            <property>
-              <name>listener</name>
-              <value>org.apache.hadoop.test.TimedOutTestsListener</value>

Review Comment:
   Taken your route. The META-INF/services descriptor is gone and the listener 
property stays in all eight poms — the PR now touches no pom at all.
   
   TimedOutTestsListener implements TestExecutionListener, so the wiring 
already in the tree starts working on a Surefire bump alone once #3438 lands. 
Dormant until then, which the description now says; the dump Hadoop's tests get 
today comes from waitFor calling dumpForTimeout directly, which needs no 
registration.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to