Graph Options correction, Clone Implementation on public structs #51
Conversation
ManevilleF
changed the title
There was a problem hiding this comment.
@fMeow So a small PR fixing a few Graph options, adding Clone to public structs and fixes requested by the CI
| - [X] Graph Management | ||
| - [X] Index Management | ||
| - [ ] User Management | ||
| - (X) Graph Management |
There was a problem hiding this comment.
For a weird reason the CI failed on this, considering it as links
| //! the retrieval of existing indexes, as ArangoDB creates a primary index on | ||
| //! every collection. | ||
| //! For detailed information about ArangoDB indexes, please check out the | ||
| //! official ArangoDB [documentation](https://www.arangodb.com/docs/stable/http/indexes.html). | ||
| //! | ||
| //! [Primary]: https://www.arangodb.com/docs/stable/http/indexes.html#primary-index |
There was a problem hiding this comment.
The CI was failing on this too, a missing link
| assert_eq!(result.orphan_collections.len(), 1); | ||
| // Would work only with Enterprise Edition | ||
| // | ||
| // assert_eq!(result.is_disjoint.unwrap(), false); |
There was a problem hiding this comment.
I can't really test this. I don't have enteprise edition and ArangoDB doesn't respond with the optional fields
There was a problem hiding this comment.
I signaled the issue to the arangoDB team and agree that it should respond, so maybe in the future it will work
|
Can someone review this? @fMeow or @inzanez, I finished my migration/schema generation system for ArangoDB (aragog and aragog_cli) But the graph part won't work perfectly until this is deployed. |
No description provided.