Skip to content

Conversation

@jitokim
Copy link
Contributor

@jitokim jitokim commented Oct 30, 2024

@jitokim jitokim changed the title GH-1623 fix typo for adding document in Neo4jVectorStore GH-1623 fix typo for updating node in Neo4jVectorStore Oct 31, 2024
Define DEFAULT_TRANSACTION_SIZE for the hardcoded batch size in the delete query

Signed-off-by: jitokim <[email protected]>
@jitokim jitokim force-pushed the gh-1623-fix-neo4j-query branch from 2f9e69d to 885b397 Compare October 31, 2024 12:11
@tzolov
Copy link
Contributor

tzolov commented Nov 3, 2024

Hi @michael-simons , can you please review this PR?

Copy link
Contributor

@michael-simons michael-simons left a comment

Choose a reason for hiding this comment

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

Nice change, also with the tx size. Added one small change request, though (OCD kicking in…)

@jitokim
Copy link
Contributor Author

jitokim commented Nov 5, 2024

@michael-simons thanks for review 😀

@markpollack markpollack self-assigned this Nov 5, 2024
@markpollack markpollack added this to the 1.0.0-M4 milestone Nov 5, 2024
@markpollack
Copy link
Member

thanks for the contribution and the review!

merged in 1cdec7b

@markpollack markpollack closed this Nov 6, 2024
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.

if the embedding property of a node is not using the default name, documents cannot be added to the vector store

4 participants