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.

Sign up for GitHub

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

Jump to bottom

Redundant port specification with embedded DB #25

Closed
dandv opened this issue Mar 27, 2023 · 1 comment
Closed

Redundant port specification with embedded DB #25

dandv opened this issue Mar 27, 2023 · 1 comment

Comments

@dandv
Copy link
Contributor

dandv commented Mar 27, 2023 •
edited

In the embedded example, the port needs to be specified twice. Could this be simplified?

The Python client disallows specifying both the URL and embedded_options.
@parkerduckworth
Copy link
Member

The port is specified because it is possible to have multiple embedded instances running per host. It can be omitted, and a default port of 6666 will be used.

Setting the host for the EmbeddedOptions will be disallowed however, because the embedded db will run on the same host as the client.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dandv @parkerduckworth