[GitHub] [incubator-druid] ccaominh opened a new pull request #8230: Fix dependency analyze warnings

[GitBox <gi...@apache.org>]

ccaominh opened a new pull request #8230: Fix dependency analyze warnings
URL: https://github.com/apache/incubator-druid/pull/8230
 
 
   ### Description
   
   Update the maven dependency plugin to the latest version and fix all
   warnings for unused declared and used undeclared dependencies in the
   compile scope. Added new travis job to add the check to CI. Also fixed
   some source code files to use the correct packages for their imports and
   updated druid-forbidden-apis to prevent regressions.
   
   **Note:** https://github.com/apache/incubator-druid/pull/8128 previously attempted to make this change, but was reverted as it resulted in several runtime errors. This second attempt changes the `scope` of several dependencies to `runtime` instead of removing them entirely. Some options were also added to the new travis job to make it run faster.
   
   <hr>
   
   This PR has:
   - [x] been self-reviewed.
   - [x] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
   - [x] added travis test.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org