Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include "--per-page=200" in all reindex examples? #307

Open
kadamwhite opened this issue Oct 26, 2021 · 1 comment
Open

Include "--per-page=200" in all reindex examples? #307

kadamwhite opened this issue Oct 26, 2021 · 1 comment

Comments

@kadamwhite
Copy link

Reading the documentation at https://docs.altis-dxp.com/search/reindexing-content/ there are several examples given for how to reindex content on a site, none of which include a --per-page argument. Then, at the end, we recommend --per-page=200 for performance reasons.

When using CLI to perform re-indexes, it's recommended to set the --per-page argument to a figure around 200. By default, this figure is set to 350 which will often cause service timeouts.

Shouldn't we include --per-page in the examples above, and explain why it's necessary at the end? I've been copying and pasting the first few examples off of this page for months and didn't really see the note about page size until now.

@roborourke
Copy link
Contributor

I believe we can actually set this default in code now too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants