[GitHub] [incubator-opendal] suyanhanx commented on a diff in pull request #2311: docs(bidnings/java): how to deploy snapshots

["suyanhanx (via GitHub)" <gi...@apache.org>]

suyanhanx commented on code in PR #2311:
URL: https://github.com/apache/incubator-opendal/pull/2311#discussion_r1204449869


##########
bindings/java/README.md:
##########
@@ -1,18 +1,95 @@
 # OpenDAL Java Bindings
 
-## Build and test
+## Build
 
-Following the command below to build `opendal-java` and run the tests:
+This project provide OpenDAL Java bindings with artifact name `opendal-java`. It depends on JDK 8 or later.

Review Comment:
   ```suggestion
   This project provides OpenDAL Java bindings with artifact name `opendal-java`. It depends on JDK 8 or later.
   ```



##########
bindings/java/README.md:
##########
@@ -1,18 +1,95 @@
 # OpenDAL Java Bindings
 
-## Build and test
+## Build
 
-Following the command below to build `opendal-java` and run the tests:
+This project provide OpenDAL Java bindings with artifact name `opendal-java`. It depends on JDK 8 or later.
+
+Thanks to [rust-maven-plugin](https://github.com/questdb/rust-maven-plugin) with can use Maven to build both Rust dynamic lib and JAR files with one command:

Review Comment:
   ```suggestion
   Thanks to [rust-maven-plugin](https://github.com/questdb/rust-maven-plugin), we can use Maven to build both Rust dynamic lib and JAR files with one command now:
   ```



-- 
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: commits-unsubscribe@opendal.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org