imbajin commented on code in PR #525:
URL: https://github.com/apache/hugegraph-doc/pull/525#discussion_r4255314491


##########
content/en/docs/quickstart/toolchain/hugegraph-loader.md:
##########
@@ -986,6 +990,29 @@ bin/hugegraph-loader.sh \
 
 Expected result: two vertices and zero edges imported. This example uses 
dedicated schema names to avoid conflicts with a preloaded `person` label. In 
Hubble, run `g.V().hasLabel('docs_smoke_person').values('docs_name')` to 
confirm `docs-smoke-alice` and `docs-smoke-bob`.
 
+#### 3.4.6 Concurrent imports and LOADING ownership {#loading-ownership}
+
+Upgrade both Server and Client/Loader to builds containing `/mode/owners` 
protocol 1; a version number or Docker `latest` tag alone does not confirm 
support.
+Server 1.7 recognizes `LOADING` but has no ownership endpoint. Normal imports 
fail if the capability is absent or invalid, without falling back to enum
+mode PUT or `NONE`. This check precedes `--clear-all-data`, Schema creation 
and import-data writes; `--create-graph` can create the target graph first.

Review Comment:
   Fixed in the current compact proposal. Both EN/ZH Loader and GraphMode pages 
label the ownership contract as proposed and unreleased, explain that released 
packages and current Apache master lack it, and link the matching Server and 
Client/Loader PR builds. Hubble task cleanup is a separate change.
   
   The current compatibility contract preserves legacy ordinary-import behavior 
only when the ownership probe returns an unsupported-route 404. Permission, 
network, invalid-response and acquisition errors remain failures. Graph-source 
and restore imports keep their existing RESTORING enum flow, without an 
ownership probe or a new automatic reset to NONE.
   
   Recovery requires stopping the import and retrying the REST release with the 
same retained instance/epoch/token and authenticated identity; Loader shutdown 
does not promise automatic retries. The protection covers one live Server graph 
instance through a fixed direct endpoint. The current paired source and both 
language versions were checked in the independent documentation review. 
Current-head CI and coordinated merges remain open.



-- 
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]

Reply via email to