You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hugegraph.apache.org by "Copilot (via GitHub)" <gi...@apache.org> on 2026/01/19 12:04:30 UTC
Re: [PR] Optimize dynamic creation of diagram documents [incubator-hugegraph-doc]
Copilot commented on code in PR #444:
URL: https://github.com/apache/incubator-hugegraph-doc/pull/444#discussion_r2704487580
##########
content/en/docs/quickstart/hugegraph/hugegraph-server.md:
##########
@@ -216,10 +216,26 @@ task.scheduler_type=distributed
pd.peers=127.0.0.1:8686,127.0.0.1:8687,127.0.0.1:8688
```
+```properties
+# Full example (with authentication)
Review Comment:
The comment says "Full example" but this example is actually simpler than the previous one (single pd.peers instead of three). Consider changing this to "Simple example" or "Minimal example" to better reflect the content.
```suggestion
# Simple example (with authentication)
```
##########
content/en/docs/quickstart/hugegraph/hugegraph-server.md:
##########
@@ -216,10 +216,26 @@ task.scheduler_type=distributed
pd.peers=127.0.0.1:8686,127.0.0.1:8687,127.0.0.1:8688
```
+```properties
+# Full example (with authentication)
+gremlin.graph=org.apache.hugegraph.auth.HugeFactoryAuthProxy
+
+# Specify storage hstore
Review Comment:
The comment should say "Specify storage backend hstore" instead of "Specify storage hstore" for grammatical correctness.
```suggestion
# Specify storage backend hstore
```
##########
content/en/docs/clients/restful-api/graphs.md:
##########
@@ -167,6 +167,8 @@ POST http://localhost:8080/graphspaces/DEFAULT/graphs/hugegraph2
##### Request Body
Create a non-auth graph (set `Content-Type: application/json`)
+**Note**!! For version 1.7.0 and earlier, if the backend is hstore, you must add "task.scheduler_type": "distributed" in
Review Comment:
Missing a space after "**Note**!!" - should be "**Note**!! For version" with proper spacing. Also consider reducing the exclamation marks to one for a more professional tone.
```suggestion
**Note**! For version 1.7.0 and earlier, if the backend is hstore, you must add "task.scheduler_type": "distributed" in
```
##########
content/en/docs/clients/restful-api/graphs.md:
##########
@@ -142,10 +142,10 @@ Clone a `non-auth` mode graph (set `Content-Type: application/json`)
##### Response Body
-```javascript
+```json
{
- "name": "hugegraph_clone",
- "backend": "rocksdb"
+ "name": "cloneGraph",
Review Comment:
Trailing space detected after "cloneGraph". Remove the trailing whitespace for consistency.
```suggestion
"name": "cloneGraph",
```
--
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: issues-unsubscribe@hugegraph.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@hugegraph.apache.org
For additional commands, e-mail: issues-help@hugegraph.apache.org