RuiLi8080 opened a new pull request #3298:
URL: https://github.com/apache/storm/pull/3298


   ## What is the purpose of the change
   
   We are seeing Mockito related error building topology with latest storm 
version.
   ```
   Configuring TestNG with: TestNG652Configurator
   SLF4J: Class path contains multiple SLF4J bindings.
   SLF4J: Found binding in 
[jar:file:/Users/rli01/.m2/repository/org/slf4j/slf4j-log4j12/1.7.10/slf4j-log4j12-1.7.10.jar!/org/slf4j/impl/StaticLoggerBinder.class]
   SLF4J: Found binding in 
[jar:file:/Users/rli01/.m2/repository/org/apache/logging/log4j/log4j-slf4j-impl/2.11.2/log4j-slf4j-impl-2.11.2.jar!/org/slf4j/impl/StaticLoggerBinder.class]
   SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an 
explanation.
   SLF4J: Actual binding is of type [org.slf4j.impl.Log4jLoggerFactory]
   Tests run: 37, Failures: 12, Errors: 0, Skipped: 0, Time elapsed: 1.135 sec 
<<< FAILURE! - in TestSuite
   testCycling(fetl.bullet.propane.PropaneSpoutTest)  Time elapsed: 0.147 sec  
<<< FAILURE!
   java.lang.ExceptionInInitializerError: null
        at 
org.mockito.internal.configuration.plugins.Plugins.getStackTraceCleanerProvider(Plugins.java:17)
        at 
org.mockito.internal.exceptions.stacktrace.StackTraceFilter.<clinit>(StackTraceFilter.java:21)
        at 
org.mockito.internal.exceptions.stacktrace.ConditionalStackTraceFilter.<init>(ConditionalStackTraceFilter.java:17)
        at 
org.mockito.exceptions.base.MockitoException.filterStackTrace(MockitoException.java:41)
        at 
org.mockito.exceptions.base.MockitoException.<init>(MockitoException.java:30)
        at 
org.mockito.exceptions.misusing.MockitoConfigurationException.<init>(MockitoConfigurationException.java:18)
        at 
org.mockito.internal.configuration.plugins.PluginLoader.loadImpl(PluginLoader.java:66)
        at 
org.mockito.internal.configuration.plugins.PluginLoader.loadPlugin(PluginLoader.java:24)
        at 
org.mockito.internal.configuration.plugins.PluginRegistry.<init>(PluginRegistry.java:12)
        at 
org.mockito.internal.configuration.plugins.Plugins.<clinit>(Plugins.java:11)
        at org.mockito.internal.util.MockUtil.<clinit>(MockUtil.java:24)
        at org.mockito.internal.MockitoCore.<init>(MockitoCore.java:44)
        at org.mockito.Mockito.<clinit>(Mockito.java:975)
        at 
fetl.bullet.propane.PropaneSpoutTest.createKafkaConsumer(PropaneSpoutTest.java:71)
        at 
fetl.bullet.propane.PropaneSpoutTest.createKafkaConsumer(PropaneSpoutTest.java:87)
        at 
fetl.bullet.propane.PropaneSpoutTest.testCycling(PropaneSpoutTest.java:170)
   ```
   
   After making the changes in this PR, topology build succeeded with latest 
local storm dependencies.
   
   ## How was the change tested
    
   Build storm locally and rebuild topology depending on it.


----------------------------------------------------------------
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.

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


Reply via email to