imbajin commented on code in PR #503: URL: https://github.com/apache/hugegraph-doc/pull/503#discussion_r4126133248
########## content/en/docs/clients/restful-api/vertex.md: ########## @@ -367,14 +367,20 @@ PUT http://127.0.0.1:8080/graphspaces/DEFAULT/graphs/hugegraph/graph/vertices/"1 #### Params - label: Vertex type -- properties: Property key-value pairs (precondition: indexes are created for property queries) +- properties: Property key-value pairs (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(18)"}` will be interpreted as an exact match, i.e., the age property is equal to the string "P.gt(18)" - offset: Offset, default is 0 - limit: Maximum number of results, default is 100 - page: Page number All of the above parameters are optional. `page` can not be combined with a non-zero `offset`, everything else can be combined in any way. +For a `properties` query, the specified `label` needs a property index compatible with the query conditions. If `label` is omitted, every Review Comment: 🧹 The blanket index requirement misses the PRIMARY_KEY fast path. When the selected vertex label uses PRIMARY_KEY and properties supplies every primary-key field, GraphTransaction.optimizeQuery() builds an IdQuery; any additional property predicates are filtered after that direct lookup. Please qualify this rule with the complete-primary-key exception and mirror it in the Chinese page. -- 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]
