returnToInnocence commented on code in PR #333:
URL: 
https://github.com/apache/incubator-hugegraph-doc/pull/333#discussion_r1618094519


##########
content/en/docs/quickstart/hugegraph-hubble.md:
##########
@@ -135,7 +116,7 @@ git clone https://github.com/apache/hugegraph-toolchain.git
 Compile `hubble`. It depends on the loader and client, so you need to build 
these dependencies in advance during the compilation process (you can skip this 
step later).
 
 ```shell
-cd incubator-hugegraph-toolchain
+cd hugegraph-toolchain
 sudo pip install -r 
hugegraph-hubble/hubble-dist/assembly/travis/requirements.txt
 mvn install -pl hugegraph-client,hugegraph-loader -am 
-Dmaven.javadoc.skip=true -DskipTests -ntp
 cd hugegraph-hubble

Review Comment:
   This optimization has been done as follows: 
   mvn -e package -Dmaven.javadoc.skip=true -Dmaven.test.skip=true -ntp
   
   It will take advantage of Maven's lifecycle mechanisms and may start 
directly from the build phase (if the project has already been compiled and the 
source code has not changed) and skip the build phase.



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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to