JaeGeunBang opened a new pull request #9562: Remove unnecessary apis
URL: https://github.com/apache/druid/pull/9562
 
 
   ### Description
   
   When building with `mvn clean test`, the following WARNING message is 
displayed for each project test.
   
   ```
   [INFO] Reading API signatures: 
/mnt/c/Users/jgb71/git/druid/codestyle/joda-time-forbidden-apis.txt
   [INFO] Reading API signatures: 
/mnt/c/Users/jgb71/git/druid/codestyle/druid-forbidden-apis.txt
   [WARNING] Method not found while parsing signature: 
com.google.common.util.concurrent.MoreExecutors#newDirectExecutorService() 
[signature ignored]
   [WARNING] Method not found while parsing signature: 
com.google.common.util.concurrent.MoreExecutors#directExecutor() [signature 
ignored]
   [WARNING] Method not found while parsing signature: 
java.nio.file.Files#createTempDirectory(java.lang.String 
prefix,java.nio.file.FileAttribute...) [signature ignored]
   [WARNING] Method not found while parsing signature: 
org.apache.commons.io.FileUtils#deleteDirectory() [signature ignored]
   [WARNING] Some signatures were ignored because the following classes were 
not found on classpath:
   [WARNING]   com.amazonaws.annotation.GuardedBy, 
com.google.common.io.BaseEncoding.base64,... (and 2 more).
   [INFO] Loading classes to check...
   [INFO] Scanning classes for violations...
   ```
   
   Even if the WARNING message is shown, it does not affect the build, but I 
think it's wrong to see the WARNING message for each project build. Methods 
that were not found after reading the API signatures were removed. So the 
warning message was no longer appear.
   
   <hr>
   
   This PR has:
   - [x] been self-reviewed.

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


With regards,
Apache Git Services

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

Reply via email to