doc : add ElasticsearchVectorStore doc #397
Conversation
github-actions
bot
added
priority/major
|
@twosom Please add the following content to your PR description and select a checkbox: |
twosom
force-pushed
the
doc-elasticsearch-vectorstore
branch
from
3e293e7 to
27bd922
Compare
github-actions
bot
added
doc-not-needed
|
|
||
| {{< hint info >}} | ||
| Vector stores are currently supported in the Python API only. Java API support is planned for future releases. | ||
| Vector stores are supported in both Python and Java APIs. |
There was a problem hiding this comment.
Maybe we can just remove this hint?
| List<Document> documents = event.getDocuments(); | ||
| // Process the retrieved documents... | ||
| } | ||
| } |
There was a problem hiding this comment.
Part of this example (i.e., how to search in the vector store with built-in events) is not specific to ES. We may consider modifying the Getting Started section and describe how to do the steps in Java. Then in the Elasticsearch section example, we can keep only the ES specific contents (i.e. creating the resource descriptor). Imagine that in future we may support many vector stores, so it might be better to avoid repeated contents as much as possible. WDYT?
| {{< /tab >}} | ||
|
|
||
| {{< /tabs >}} | ||
|
|
There was a problem hiding this comment.
Shall we also add instructions for building custom vector stores in Java?
2 participants
Linked issue: #385, #321
Purpose of change
Added documentation for ElasticsearchVectorStore.
Tests
API
Documentation
doc-neededdoc-not-needed