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

Update multistore-setup-options.md #1907

Merged
merged 1 commit into from
Jun 7, 2023
Merged

Conversation

helen-laktionova
Copy link
Collaborator

Changed search to Elasticsearch

PR Description

TBD

Checklist

  • I agree with the Code Contribution License Agreement in CONTRIBUTING.md

Changed search to Elasticsearch
@@ -189,7 +189,7 @@ The following tables contain high-level criteria that sum up the setups describe
| What | Setup 1 | Setup 2 | Setup 2 |
|------------------------------------------------------------------------|---------|----------|----------|
| DB | Shared | Separate | Separate |
| Ke-value storage (Redis) and search (OpenSearch/ElasticCache) services | Shared | Shared | Separate |
| Ke-value storage (Redis) and Elasticsearch (OpenSearch/ElasticCache) services | Shared | Shared | Separate |
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ke-value -> Key value

@VadymSachenko VadymSachenko added small changes PR contains small changes, like typo fixes ready for merge The PR can be merged labels Jun 7, 2023
@VadymSachenko VadymSachenko merged commit 2d64ee6 into master Jun 7, 2023
@VadymSachenko VadymSachenko deleted the helen-laktionova-patch-2 branch June 7, 2023 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for merge The PR can be merged small changes PR contains small changes, like typo fixes
Development

Successfully merging this pull request may close these issues.

2 participants