czy006 commented on code in PR #3306:
URL: https://github.com/apache/amoro/pull/3306#discussion_r1832143925


##########
README.md:
##########
@@ -117,6 +117,7 @@ Amoro is built using Maven with JDK 8 and JDK 17(only for 
`amoro-format-mixed/am
 * Build and skip tests: `mvn clean package -DskipTests`
 * Build and skip dashboard: `mvn clean package -Pskip-dashboard-build`
 * Build and disable disk storage, RocksDB will NOT be introduced to avoid 
memory overflow: `mvn clean package -DskipTests -Pno-extented-disk-storage`
+* Build and disable aliyun sdk: `mvn clean package -DskipTests -Pno-aliyun-sdk`

Review Comment:
   I recommend not compiling **Aliyun OSS** by default, if the user wants to 
use it can use **-Paliyun-oss-sdk** 



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