You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by GitBox <gi...@apache.org> on 2021/05/13 08:00:04 UTC
[GitHub] [ambari] martin-g opened a new pull request #3308: Fix import order. Fixes Checkstyle error at build time
martin-g opened a new pull request #3308:
URL: https://github.com/apache/ambari/pull/3308
## What changes were proposed in this pull request?
This PR changes the order of imports in a .java file.
Currently the build breaks with:
```
INFO] --- maven-checkstyle-plugin:2.17:check (checkstyle) @ ambari-server ---
[INFO] Starting audit...
[ERROR] /home/ubuntu/git/apache/ambari/ambari-server/src/main/java/org/apache/ambari/server/orm/entities/HostStateEntity.java:36: Wrong order for 'java.util.Objects' import. [ImportOrder]
Audit done.
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Ambari Admin View 3.0.0.0-SNAPSHOT ................. SUCCESS [ 8.914 s]
[INFO] ambari-utility 3.0.0.0-SNAPSHOT .................... SUCCESS [01:28 min]
[INFO] Ambari Server SPI 3.0.0.0-SNAPSHOT ................. SUCCESS [ 5.731 s]
[INFO] Ambari Service Advisor 1.0.0.0-SNAPSHOT ............ SUCCESS [ 3.070 s]
[INFO] Ambari Server 3.0.0.0-SNAPSHOT ..................... FAILURE [13:54 min]
[INFO] Ambari Functional Tests 3.0.0.0-SNAPSHOT ........... SKIPPED
[INFO] Ambari Agent 3.0.0.0-SNAPSHOT ...................... SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 15:40 min
[INFO] Finished at: 2021-05-13T08:59:12+01:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:2.17:check (checkstyle) on project ambari-server: Failed during checkstyle execution: There is 1 error reported by Checkstyle 8.9 with /home/ubuntu/git/apache/ambari/ambari-server/checkstyle.xml ruleset. -> [Help 1]
[ERROR]
```
## How was this patch tested?
The build passed after this change.
--
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
[GitHub] [ambari] martin-g commented on pull request #3308: Fix import order. Fixes Checkstyle error at build time
Posted by GitBox <gi...@apache.org>.
martin-g commented on pull request #3308:
URL: https://github.com/apache/ambari/pull/3308#issuecomment-840397253
It seems there is PR already: https://github.com/apache/ambari/pull/3295
--
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
[GitHub] [ambari] martin-g closed pull request #3308: Fix import order. Fixes Checkstyle error at build time
Posted by GitBox <gi...@apache.org>.
martin-g closed pull request #3308:
URL: https://github.com/apache/ambari/pull/3308
--
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