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

Refactoring RemoteDatabase API #1327

Closed
lvca opened this issue Nov 16, 2023 · 1 comment
Closed

Refactoring RemoteDatabase API #1327

lvca opened this issue Nov 16, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@lvca
Copy link
Contributor

lvca commented Nov 16, 2023

With ArcadeDB 23.10.1, the RemoteDatabase API has 2 use cases:

  • manage the server
  • being a BasicDatabase implementation to emulate most of the EmbeddedDatabase API

The ideal would be splitting the class in 2:

  • RemoteDatabase as only a BasicDatabase implementation that works with a remote server and
  • RemoteServer, as a new class that allows to control the server, namely: create/drop databases, create/drop users, list for available databases and in the future more operations with the server. This class doesn't need the database name

This would break the current API for whom is using RemoteDatabase to manage server users, list the available databases and create new databases.

@lvca lvca added the enhancement New feature or request label Nov 16, 2023
@lvca lvca added this to the 23.11.1 milestone Nov 16, 2023
@lvca lvca self-assigned this Nov 16, 2023
lvca added a commit that referenced this issue Nov 16, 2023
@lvca
Copy link
Contributor Author

lvca commented Nov 16, 2023

Completed. @gramian, I moved the server methods in the new RemoteServer class for documentation purposes.

@lvca lvca closed this as completed Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant