imbajin commented on code in PR #523: URL: https://github.com/apache/hugegraph-doc/pull/523#discussion_r4254753947
########## content/en/docs/config/config-backend-hstore.md: ########## @@ -246,3 +246,17 @@ curl http://localhost:8080/metrics/backend The `nodes` value in the response is the count of active stores PD returns. A `nodes` value of `0` means the Server reached PD but PD has no store in state `Up`, which usually means the Store nodes have not registered yet, or registered as `Pending` because they are not in PD's `pd.initial-store-list`. + +Store builds containing the legacy GraphID preflight check existing graph-data column families before allocating a missing graph ID, first opening a cold +partition DB during metadata bootstrap, starting its Raft state machine, or activating a staged snapshot. Before pruning, the first cold DB open checks +all retained valid versions. Reserved `FFFE` legacy rows cause refusal. +An incomplete selected DB version without `CURRENT` is also rejected when it would hide retained valid versions. This concerns affected old write paths; +it does not mean that every 1.7 database needs migration, and the persisted ID format is unchanged. + +On refusal, preserve the original data DB versions, metadata, WAL and incoming/staged snapshots. Investigate the affected write and upgrade history before +resuming writes or replay. Do not delete graph-to-ID mappings or assign all reserved-prefix rows to one graph: different graphs may share that prefix. +Store/Raft location metadata may already have been updated before the check, so the entire bootstrap is not read-only. The check performs no automatic +migration or ownership repair; a complete 1.7 upgrade and recovery still require deployment-specific validation. + +See the [preflight change](https://github.com/hugegraph/hugegraph/pull/301) and the Review Comment: Addressed in [the bilingual documentation correction](https://github.com/apache/hugegraph-doc/commit/0f296459c2adcd515429c04857d875b2fff289ab). Both pages now link the proposed [Apache HugeGraph #3330](https://github.com/apache/hugegraph/pull/3330) implementation, explicitly state that it is unmerged and absent from Apache releases, and limit the refusal behavior to builds containing that fix. The guide is pinned to the proposed revision. Coordinated code/docs merge and complete deployment upgrade/recovery acceptance remain open. ########## content/en/docs/config/config-backend-hstore.md: ########## @@ -246,3 +246,17 @@ curl http://localhost:8080/metrics/backend The `nodes` value in the response is the count of active stores PD returns. A `nodes` value of `0` means the Server reached PD but PD has no store in state `Up`, which usually means the Store nodes have not registered yet, or registered as `Pending` because they are not in PD's `pd.initial-store-list`. + +Store builds containing the legacy GraphID preflight check existing graph-data column families before allocating a missing graph ID, first opening a cold Review Comment: Fixed in [the bilingual documentation correction](https://github.com/apache/hugegraph-doc/commit/0f296459c2adcd515429c04857d875b2fff289ab). The English and Chinese pages now have their own numbered “9 Legacy GraphID upgrade preflight” / “9 旧 GraphID 升级预检” sections. Strict Hugo builds, link checks and rendered heading/outline readback passed for both languages. -- 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]
