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


##########
content/en/docs/clients/restful-api/edge.md:
##########
@@ -416,12 +416,25 @@ It is not possible to delete an attribute that is not set 
as nullable.
 - vertex_id: Vertex ID
 - direction: Edge direction (OUT | IN | BOTH), default is BOTH
 - label: Edge label
-- properties: Key-value pairs of properties (requires pre-built indexes for 
property queries)
+- properties: Key-value pairs of properties (see index coverage below)
 - keep_start_p: Default is false. When set to true, the range matching input 
expression will not be automatically escaped. For example, 
`properties={"age":"P.gt(0.8)"}` will be interpreted as an exact match, i.e., 
the age property is equal to "P.gt(0.8)"
 - offset: Offset, default is 0
 - limit: Number of queries, default is 100
 - page: Page number
 
+Without `vertex_id`, a `properties` query searches edges globally. The 
specified `label` needs a property index compatible with the query conditions. 
If `label`
+is omitted, candidate labels are query-visible edge labels that can hold edges 
and declare all queried properties, including labels that currently have no
+edges. Labels hidden from normal query results, including those being deleted, 
do not require coverage. A base (parent) edge label has no direct edges and does
+not need a separate index when its sub-labels are covered. A compatible parent 
index can cover its sub-labels; compatible indexes on every candidate sub-label

Review Comment:
   🧹 Blocking: no. Summary: The parent-index alternative omits the parent's 
visibility requirement. In the paired server change, hidden or deleting parents 
are excluded before indexes are collected, so a visible child without its own 
compatible index still fails coverage; please qualify the sentence and mirror 
it in Chinese. Evidence: apache/hugegraph#3243 at 7f5a917; 
GraphIndexTransaction.collectMatchedIndexes() filters labels using 
rootOriginQuery() before checkIndexCoverage().



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