github-actions[bot] commented on code in PR #69574: URL: https://github.com/apache/doris/pull/69574#discussion_r4367080607
########## fe/fe-core/src/test/java/org/apache/doris/cloud/datasource/CloudInternalCatalogTest.java: ########## @@ -52,6 +52,8 @@ import org.mockito.MockedStatic; import org.mockito.Mockito; +import doris.segment_v2.SegmentV2; Review Comment: [P2] Keep the generated-proto import in alphabetical order `doris.segment_v2.SegmentV2` is in the same third-party import group as the JUnit/Mockito imports, but it is added after them with an extra blank line. `fe/check/checkstyle/checkstyle.xml` enables alphabetical ordering within that group, and `fe/pom.xml` runs the error-level check during `validate` with test sources included. Consequently the standard FE Maven validation fails on this new import. Move it before `org.junit.jupiter.api.AfterEach` in the same import block. -- 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]
