Skip to content

Dev memory config vector indexes #2289

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

Merged

Conversation

renetapopova
Copy link
Collaborator

No description provided.

@mauvo
Copy link
Contributor

mauvo commented Apr 29, 2025

  • Let's add a link back to the vector index documentation. And we need a link from the vector index documentation to the memory config page.

I think this is clearer, that we're agreeing with the memory config section, not contradicting it.
@mauvo
Copy link
Contributor

mauvo commented Apr 29, 2025

  • I think "Warming up the vector index", should be moved to below the content that is directly discussing memory configuration. Probably it should be moved to the bottom of the page.

mauvo and others added 4 commits April 29, 2025 18:23
I think we mean that the machine must have sufficient memory. Saying OS could be confused with the OS memory we say Lucene requires.
@renetapopova
Copy link
Collaborator Author

@mauvo, I think it looks good. Shall I merge it?

@neo4j-docops-agent
Copy link
Collaborator

neo4j-docops-agent commented May 6, 2025

Thanks for the documentation updates.

The preview documentation has now been torn down - reopening this PR will republish it.

@renetapopova renetapopova merged commit 26b44a3 into neo4j:dev May 7, 2025
8 checks passed
@renetapopova renetapopova deleted the dev-memory-config-vector-indexes branch May 7, 2025 08:39
renetapopova added a commit to renetapopova/docs-operations that referenced this pull request May 7, 2025
renetapopova added a commit to renetapopova/docs-operations that referenced this pull request May 7, 2025
renetapopova added a commit that referenced this pull request May 7, 2025
renetapopova added a commit that referenced this pull request May 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants