jackylee-ch opened a new pull request, #9049:
URL: https://github.com/apache/paimon/pull/9049

   ### Purpose
   
   `spotless:check` currently fails on master. In 
`TestChangelogDataReadWrite#createRead`, the
   `KeyValueTableRead` constructor call is split across multiple lines, but 
google-java-format (AOSP)
   fits it on one line:
   
   ```java
   -        return new KeyValueTableRead(
   -                () -> read,
   -                () -> rawFileRead,
   -                schema,
   -                options,
   -                null);
   +        return new KeyValueTableRead(() -> read, () -> rawFileRead, schema, 
options, null);
   ```
   
   Since `Check licensing` runs spotless first and most other jobs depend on 
it, every open PR that is
   merged against current master goes red regardless of its own content.
   
   The change is exactly the output of `mvn -pl 
paimon-flink/paimon-flink-common spotless:apply` — one
   file, 1 insertion, 6 deletions, no behavior change.
   
   ### Tests
   
   `FileStoreSourceReaderTest` and `FileStoreSourceSplitReaderTest` (the tests 
using this helper) pass:
   18/18. `spotless:check` and `checkstyle:check` are clean on the module.
   


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

Reply via email to