Copilot commented on code in PR #2970:
URL: https://github.com/apache/hugegraph/pull/2970#discussion_r2938995922
##########
hugegraph-store/README.md:
##########
@@ -214,6 +214,7 @@ Start the Store server:
```bash
# Replace {version} with your hugegraph version
+# Note: historical releases <= 1.7.0 still use the "-incubating" directory name
Review Comment:
The added note says releases <= 1.7.0 use a `-incubating` directory name,
but the command directly below uses `cd apache-hugegraph-store-{version}` (no
`-incubating`). Please make the guidance copy/pasteable by either correcting
the cutoff/version claim or showing the actual directory naming pattern for
those historical releases.
##########
hugegraph-store/docs/deployment-guide.md:
##########
@@ -626,6 +628,7 @@ curl http://192.168.1.10:8620/v1/stores
```bash
# Extract Server distribution
+# Note: historical releases <= 1.7.0 still use the "-incubating"
package/directory names
tar -xzf apache-hugegraph-1.7.0.tar.gz
cd apache-hugegraph-1.7.0
Review Comment:
Same inconsistency in the Server section: the note claims `-incubating`
naming for <= 1.7.0, but the example uses `apache-hugegraph-1.7.0(.tar.gz)` /
`cd apache-hugegraph-1.7.0` without it. Please reconcile these so the
deployment guide remains correct for the versions it references.
##########
hugegraph-store/docs/deployment-guide.md:
##########
@@ -509,6 +510,7 @@ curl http://192.168.1.10:8620/v1/members
```bash
# Extract Store distribution
+# Note: historical releases <= 1.7.0 still use the "-incubating"
package/directory names
tar -xzf apache-hugegraph-store-1.7.0.tar.gz
cd apache-hugegraph-store-1.7.0
Review Comment:
Same issue here: the note says releases <= 1.7.0 use `-incubating` names,
but the example uses `apache-hugegraph-store-1.7.0(.tar.gz)` without the
suffix. Please align the note and the example so users can follow the command
as written.
##########
hugegraph-store/docs/deployment-guide.md:
##########
@@ -416,6 +416,7 @@ df -h
```bash
# Extract PD distribution
+# Note: historical releases <= 1.7.0 still use the "-incubating"
package/directory names
tar -xzf apache-hugegraph-pd-1.7.0.tar.gz
cd apache-hugegraph-pd-1.7.0
Review Comment:
This note states releases <= 1.7.0 use `-incubating` package/directory
names, but the example directly below is `apache-hugegraph-pd-1.7.0.tar.gz` /
`cd apache-hugegraph-pd-1.7.0` (no `-incubating`). Please reconcile the note
with the concrete 1.7.0 example (either adjust the cutoff or update the example
names) so the instructions remain accurate.
##########
README.md:
##########
@@ -223,6 +223,7 @@ Download pre-built packages from the [Download
Page](https://hugegraph.apache.or
```bash
# Download and extract
+# Note: historical releases <= 1.7.0 still use the "-incubating" package name
Review Comment:
This note conflicts with the example commands immediately below: for version
1.7.0 the file/directory names shown here are `apache-hugegraph-1.7.0(.tar.gz)`
(no `-incubating`). Either update the note to the correct cutoff version, or
adjust the example to show the actual `-incubating` filename/directory pattern
for those historical releases (so users can copy/paste successfully).
--
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]
