Skip to content
This repository was archived by the owner on Dec 13, 2023. It is now read-only.

adds arangosearch mention to indexes page #986

Merged
merged 5 commits into from
Apr 28, 2022
Merged

adds arangosearch mention to indexes page #986

merged 5 commits into from
Apr 28, 2022

Conversation

@cw00dw0rd
Copy link
Contributor

  • The ArangoSearch section isn't mentioned on the indexing page but I think it should be since it lives there now.
@Simran-B
Copy link
Collaborator

Yes, thanks!
I would word it differently, though. Views are more than inverted indexes, and I would like to avoid any confusion with the upcoming per-collection inverted indexes.
@cw00dw0rd
Copy link
Contributor Author

I agree that there is a better way to phrase it. My goal was to inform the reader about why ArangoSearch is being included in the indexing section, since it is more than just an index.
Have any suggestions?

Since the guides cover all of ArangoSearch we could just do something like

How to use ArangoSearch views, analyzers, and functions.
@ansoboleva
Copy link
Collaborator

Implemented @cw00dw0rd's suggestion.
ansoboleva
ansoboleva approved these changes Apr 28, 2022
View reviewed changes
Copy link
Collaborator

@ansoboleva ansoboleva left a comment

Choose a reason for hiding this comment

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

reviewed
@ansoboleva ansoboleva merged commit 4afd6e6 into main Apr 28, 2022
@ansoboleva ansoboleva deleted the arangosearch_indexing branch April 28, 2022 11:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

3 participants

@cw00dw0rd @Simran-B @ansoboleva