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

support multiple URLs in connection string #79

Closed
wants to merge 2 commits into from

Conversation

rursprung
Copy link
Collaborator

see individual commit messages for further details.

the underlying OpenSearch library supports multiple URLs, thus we should
do the same here as well.

since Liquibase has no concept of this on its own we have to use a
workaround and use a character to concatenate multiple URLs together.
`;` is being used for this as it's normally not part of URLs. the
`opensearch:` prefix should not be repeated.

i.e. a URL with multiple addresses should be defined like this:
```
opensearch:http://localhost:9200;http://localhost:9201;http://localhost:9203
```
the login information is the same for all, since they must form a
cluster.
@rursprung
Copy link
Collaborator Author

let's try again with a PR where the branch is in this repo (and will thus not fail in CI): #81
i've noticed that there's no issue for the failures => i raised one: #80

@rursprung rursprung closed this Nov 22, 2024
@rursprung rursprung deleted the support-multiple-urls branch November 22, 2024 14:09
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.

1 participant