javeme commented on code in PR #270:
URL:
https://github.com/apache/incubator-hugegraph-doc/pull/270#discussion_r1226821141
##########
content/en/docs/config/config-guide.md:
##########
@@ -261,9 +261,9 @@ Our system can have multiple graphs, and the backend of
each graph can be differ
The configuration method is simple:
-**Modify `rest-server.properties`**
+**[Optional]: Modify `rest-server.properties`**
-Modify the graph configuration file path (default path) in the graphs domain
of rest-server.properties, for example:
+You can modify the graph profile directory in the `graphs` field of
`rest-server.properties`. The default configuration is `graphs=. /conf/graphs`,
if you want to change it to another directory then adjust the `graphs` domain,
for example to `graphs=/etc/hugegraph/graphs`, example is as follows:
Review Comment:
prefer to unify "field" and "domain" to "option"
##########
content/cn/docs/config/config-guide.md:
##########
@@ -263,15 +263,15 @@ cassandra.password=
配置方法也很简单:
-**修改 rest-server.properties**
+**[可选]:修改 rest-server.properties**
-在 rest-server.properties 的 graphs 域中修改图的配置文件路径(默认路径),比如:
+可以在 `rest-server.properties` 的 `graphs` 域中修改图的配置文件目录。默认配置为
`graphs=./conf/graphs`,如果想要修改为其它目录则调整 `graphs` 域,比如调整为
`graphs=/etc/hugegraph/graphs`,示例如下:
Review Comment:
为了更加更加明确规范,能否把文档中的 “域” 统一为 “配置项” 呢?
##########
content/en/docs/config/config-guide.md:
##########
@@ -261,9 +261,9 @@ Our system can have multiple graphs, and the backend of
each graph can be differ
The configuration method is simple:
-**Modify `rest-server.properties`**
+**[Optional]: Modify `rest-server.properties`**
-Modify the graph configuration file path (default path) in the graphs domain
of rest-server.properties, for example:
+You can modify the graph profile directory in the `graphs` field of
`rest-server.properties`. The default configuration is `graphs=. /conf/graphs`,
if you want to change it to another directory then adjust the `graphs` domain,
for example to `graphs=/etc/hugegraph/graphs`, example is as follows:
Review Comment:
an extra space in `graphs=. /conf/graphs`
##########
content/en/docs/config/config-guide.md:
##########
@@ -261,9 +261,9 @@ Our system can have multiple graphs, and the backend of
each graph can be differ
The configuration method is simple:
-**Modify `rest-server.properties`**
+**[Optional]: Modify `rest-server.properties`**
-Modify the graph configuration file path (default path) in the graphs domain
of rest-server.properties, for example:
+You can modify the graph profile directory in the `graphs` field of
`rest-server.properties`. The default configuration is `graphs=. /conf/graphs`,
if you want to change it to another directory then adjust the `graphs` domain,
for example to `graphs=/etc/hugegraph/graphs`, example is as follows:
Review Comment:
"for example to " => "e.g. adjust it to"
--
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]