Skip to content

Conversation

@tjoubert
Copy link
Contributor

GET/_api/collection/{collection-name}/checksum Return checksum for the collection
PUT/_api/collection/{collection-name}/loadIndexesIntoMemory Load Indexes into Memory
PUT/_api/collection/{collection-name}/recalculateCount Recalculate count of a collection
PUT​/_api​/collection​/{collection-name}​/responsibleShard Return responsible shard for a document
GET/_api/collection/{collection-name}/shards Return the shard ids of a collection
PUT/_api/collection/{collection-name}/compact Compact a collection

Copy link
Collaborator

@DiscoPYF DiscoPYF left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @tjoubert , looks good. I left a few comment regarding aligning some method names with others by adding the HTTP method suffix, for concistency, and various other minor items.

Happy to merge once discussed/done.

@tjoubert
Copy link
Contributor Author

tjoubert commented Apr 6, 2022

@DiscoPYF , thanks for your review. I've made all the relevant changes :-)

Copy link
Collaborator

@DiscoPYF DiscoPYF left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thank you! I'm happy to merge pending passing CI checks.

We need to set them up again now that the repo is in a different org. I have emailed @rossmills99 and @Zyqsempai

@Zyqsempai Zyqsempai merged commit 28d6821 into ArangoDB-Community:master Apr 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants