bitflicker64 commented on PR #557: URL: https://github.com/apache/hugegraph-doc/pull/557#issuecomment-6571781200
## Review summary: R3 (FULL) **Reviewed:** head `8faa8816` against base `8f74072b`, all 108 files. Previous round: [R2 summary](https://github.com/apache/hugegraph-doc/pull/557#issuecomment-6559038492), at head `01b8113f`. **Verdict:** not ready to merge yet. Navigation, aliases and versioning are clean, and the R2 items are fixed or mostly fixed. The newly added feature content, however, has several factual errors against current upstream code. A reader who copies these configs or examples will hit failures. **Score: 0.3 / 10.** This is the mean of 2 of 4 reviewers so far, so it is provisional. Reviewer B hit the floor of 0. | Reviewer | Raw | Adjusted | Deductions | |---|---|---|---| | Reviewer A (model not reported) | 0.5 | 0.5 | 4 Important, 3 Minor | | Reviewer B (Claude Opus 5.5) | 0.0 | 0.0 | 7 Important, 7 Minor (5 provisional) | | Reviewer C | pending | | | | Reviewer D | pending | | | **Important** (all newly added text; EN and CN): - [Hubble Vermeer port `8888` should be `6688`](https://github.com/apache/hugegraph-doc/pull/557#discussion_r4439901474) - [Hubble Vermeer algorithm list](https://github.com/apache/hugegraph-doc/pull/557#discussion_r4439901488) - [LLM SDK is not on PyPI](https://github.com/apache/hugegraph-doc/pull/557#discussion_r4439901496) - [LLM SDK examples use non-existent arguments](https://github.com/apache/hugegraph-doc/pull/557#discussion_r4439901502) - [`rocksdb.provider` default and allowed values](https://github.com/apache/hugegraph-doc/pull/557#discussion_r4439901508) - [PD drain check names the wrong config key](https://github.com/apache/hugegraph-doc/pull/557#discussion_r4439901516) - [Vermeer cluster-snapshot schema](https://github.com/apache/hugegraph-doc/pull/557#discussion_r4439901521) **Minor:** - [Batch same-neighbors response fields](https://github.com/apache/hugegraph-doc/pull/557#discussion_r4439901529) - [PD `cd` glob](https://github.com/apache/hugegraph-doc/pull/557#discussion_r4439901532) - [Committed internal audit log](https://github.com/apache/hugegraph-doc/pull/557#discussion_r4439901542). Reviewers were split on this one. - [Remaining legacy Java anchors](https://github.com/apache/hugegraph-doc/pull/557#discussion_r4439903443) **[Uncertain / needs confirmation]:** reported by one reviewer and not yet confirmed: - the MCP page requires Python ≥ 3.12, while `[email protected]` supports ≥ 3.10; - the Computer page requires JDK 17 but pins 1.7.0; - the chunk `stage` list in `hugegraph-ai/rest-api.md` is incomplete; - the Python client claims LlamaIndex integration. **Checks:** `dist/validate-links.sh` passes, and a strict Hugo 0.168.0 build passes. Navigation built from the release-1.7.0 snapshot matches the expected stats. No full latest or 1.7 artifact build was done. -- 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]
