4paradigm / 4paradigm/OpenMLDB
Resolve the JavaDoc warning of hybridse-proto and openmldb-native
- 主要言語
- C++
- スター
- 1.7k
- フォーク
- 331
- 平均マージ
- 12日 12時間
- マージ済み PR(30日)
- 1
説明
Now we can get logs of warning of Java module `hybridse-proto` when compiling.
```
[INFO] --- maven-javadoc-plugin:2.9.1:jar (attach-javadocs) @ hybridse-proto ---
[INFO]
100 warnings
[WARNING] Javadoc Warnings
[WARNING] /Users/tobe/code/4pd/OpenMLDB/java/hybridse-proto/src/main/java/com/_4paradigm/hybridse/tablet/Tablet.java:2495: warning: no @return
[WARNING] boolean hasTable(); [WARNING] ^
[WARNING] /Users/tobe/code/4pd/OpenMLDB/java/hybridse-proto/src/main/java/com/_4paradigm/hybridse/tablet/Tablet.java:2499: warning: no @return
[WARNING] com._4paradigm.hybridse.type.TypeOuterClass.TableDef getTable();
[WARNING] ^
[WARNING] /Users/tobe/code/4pd/OpenMLDB/java/hybridse-proto/src/main/java/com/_4paradigm/hybridse/tablet/Tablet.java:2503: warning: no @return
[WARNING] com._4paradigm.hybridse.type.TypeOuterClass.TableDefOrBuilder getTableOrBuilder();
[WARNING] ^
[WARNING] /Users/tobe/code/4pd/OpenMLDB/java/hybridse-proto/src/main/java/com/_4paradigm/hybridse/tablet/Tablet.java:2508: warning: no @return
[WARNING] boolean hasTid();
[WARNING] ^
[WARNING] /Users/tobe/code/4pd/OpenMLDB/java/hybridse-proto/src/main/java/com/_4paradigm/hybridse/tablet/Tablet.java:2512: warning: no @return
[WARNING] int getTid();
[WARNING] ^
[WARNING] /Users/tobe/code/4pd/OpenMLDB/java/hybridse-proto/src/main/java/com/_4paradigm/hybridse/tablet/Tablet.java:2517: warning: no @return
[WARNING] java.util.List getPidsList();
[WARNING] ^
[WARNING] /Users/tobe/code/4pd/OpenMLDB/java/hybridse-proto/src/main/java/com/_4paradigm/hybridse/tablet/Tablet.java:2521: warning: no @return
[WARNING] int getPidsCount();
[WARNING] ^
[WARNING] /Users/tobe/code/4pd/OpenMLDB/java/hybridse-proto/src/main/java/com/_4paradigm/hybridse/tablet/Tablet.java:2525: warning: no @param for in
dex
[WARNING] int getPids(int index);
[WARNING] ^
[WARNING] /Users/tobe/code/4pd/OpenMLDB/java/hybridse-proto/src/main/java/com/_4paradigm/hybridse/tablet/Tablet.java:2525: warning: no @return
......
```
You can reproduce the issue by compiling java modules with `mvn clean package`.
Since the code is generated by `protobuf` and it would be better to find a way to dismiss the warning.
コントリビューションガイド
調査の方向性
The warnings are from the maven-javadoc-plugin when building the hybridse-proto module. Look at the generated Java code in java/hybridse-proto/src/main/java/com/_4paradigm/hybridse/tablet/Tablet.java. Investigate the protobuf configuration or maven plugin settings to suppress javadoc warnings for generated code. Run `mvn clean package` to reproduce, then check the pom.xml for javadoc plugin configuration.
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- java
- 領域
- build-system, documentation
- issue の種類
- ドキュメント
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 65/100