n24q02m commented on PR #499: URL: https://github.com/apache/hugegraph-doc/pull/499#issuecomment-6160162096
Thanks, all three points are addressed on the new head. - **Range predicate without a RANGE index:** the schema section now requires a `RANGE` index on `person.age` (the page gives the `personByAge` `indexLabel` statement to add to the `scripts/example.groovy` schema), says that a `WHERE` property predicate needs a covering index, and `migrate-from-neo4j.md` has a new row for it. `cypher-compatibility.md` names the fixture schema used for the 5039e5b run. EN and CN (`21083db`). - **Missing `$param` on writes:** the wording no longer says a write stores null. It now says `SET` removes the property (or fails on a non-null key) and `CREATE` stores the `" cypher.null"` marker string or fails the type check. EN and CN (`21083db`). - **160-character wrap:** the five files are rewrapped in `70a1278`. The only lines still over 160 characters are single-line links to the test sources, which cannot be split. Wording is unchanged; `dist/validate-links.sh` passes and a Hugo build of the head succeeds. -- 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]
