This is an automated email from the ASF dual-hosted git repository.
nicholasjiang pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/celeborn.git
The following commit(s) were added to refs/heads/main by this push:
new 5f298f5ce [CELEBORN-1190][FOLLOWUP] Use
-XepDisableWarningsInGeneratedCode to disable warnings for openapi-client module
5f298f5ce is described below
commit 5f298f5ce202dd98cf4f8490e2a41c09a1b9e9cb
Author: SteNicholas <[email protected]>
AuthorDate: Wed Mar 26 12:04:07 2025 +0800
[CELEBORN-1190][FOLLOWUP] Use -XepDisableWarningsInGeneratedCode to disable
warnings for openapi-client module
### What changes were proposed in this pull request?
Use `-XepDisableWarningsInGeneratedCode` to disable warnings for
`openapi-client` module.
### Why are the changes needed?
There are some warnings in compilation of `openapi-client` module as
follows:
```
$ mvn clean install -pl openapi/openapi-client -DskipTests
-Dcheckstyle.skip=true -Drat.skip=true -Dspotless.check.skip=true|grep
"[WARNING].*java.*"
[WARNING]
/Users/nicholasjiang/Github/celeborn/openapi/openapi-client/src/main/java/org/apache/celeborn/rest/v1/worker/invoker/ApiException.java:[100,18]
[OverrideThrowableToString] To return a custom message with a Throwable class,
one should override getMessage() instead of toString().
[WARNING]
/Users/nicholasjiang/Github/celeborn/openapi/openapi-client/src/main/java/org/apache/celeborn/rest/v1/model/DynamicConfig.java:[55,19]
[ImmutableEnumChecker] enums should be immutable: 'LevelEnum' has non-final
field 'value'
[WARNING]
/Users/nicholasjiang/Github/celeborn/openapi/openapi-client/src/main/java/org/apache/celeborn/rest/v1/master/invoker/ApiException.java:[100,18]
[OverrideThrowableToString] To return a custom message with a Throwable class,
one should override getMessage() instead of toString().
[WARNING]
/Users/nicholasjiang/Github/celeborn/openapi/openapi-client/src/main/java/org/apache/celeborn/rest/v1/model/WorkerExitRequest.java:[51,19]
[ImmutableEnumChecker] enums should be immutable: 'TypeEnum' has non-final
field 'value'
[WARNING]
/Users/nicholasjiang/Github/celeborn/openapi/openapi-client/src/main/java/org/apache/celeborn/rest/v1/model/PartitionLocationData.java:[58,19]
[ImmutableEnumChecker] enums should be immutable: 'ModeEnum' has non-final
field 'value'
[WARNING]
/Users/nicholasjiang/Github/celeborn/openapi/openapi-client/src/main/java/org/apache/celeborn/rest/v1/model/PartitionLocationData.java:[107,19]
[ImmutableEnumChecker] enums should be immutable: 'StorageEnum' has non-final
field 'value'
[WARNING]
/Users/nicholasjiang/Github/celeborn/openapi/openapi-client/src/main/java/org/apache/celeborn/rest/v1/model/SendWorkerEventRequest.java:[60,19]
[ImmutableEnumChecker] enums should be immutable: 'EventTypeEnum' has
non-final field 'value'
```
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Local test:
```
$ mvn clean install -pl openapi/openapi-client -DskipTests
-Dcheckstyle.skip=true -Drat.skip=true -Dspotless.check.skip=true|grep
"[WARNING].*java.*"
```
Closes #3169 from SteNicholas/CELEBORN-1190.
Authored-by: SteNicholas <[email protected]>
Signed-off-by: SteNicholas <[email protected]>
---
pom.xml | 3 +++
1 file changed, 3 insertions(+)
diff --git a/pom.xml b/pom.xml
index e5ad162d8..f9b46905b 100644
--- a/pom.xml
+++ b/pom.xml
@@ -1190,6 +1190,7 @@
<compilerArgs>
<arg>-XDcompilePolicy=simple</arg>
<arg>-Xplugin:ErrorProne \
+ -XepDisableWarningsInGeneratedCode \
-XepExcludedPaths:.*/target/generated-sources/.* \
-Xep:BadImport:OFF \
-Xep:EmptyBlockTag:OFF \
@@ -1566,6 +1567,7 @@
<compilerArgs>
<arg>-XDcompilePolicy=simple</arg>
<arg>-Xplugin:ErrorProne \
+ -XepDisableWarningsInGeneratedCode \
-XepExcludedPaths:.*/target/generated-sources/.* \
-Xep:BadImport:OFF \
-Xep:EmptyBlockTag:OFF \
@@ -1626,6 +1628,7 @@
<compilerArgs>
<arg>-XDcompilePolicy=simple</arg>
<arg>-Xplugin:ErrorProne \
+ -XepDisableWarningsInGeneratedCode \
-XepExcludedPaths:.*/target/generated-sources/.* \
-Xep:BadImport:OFF \
-Xep:EmptyBlockTag:OFF \
