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

Add Query service to the SDK #348

Closed
Zork33 opened this issue Jan 23, 2024 · 3 comments
Closed

Add Query service to the SDK #348

Zork33 opened this issue Jan 23, 2024 · 3 comments
Assignees
@Zork33
Labels
enhancement New feature or request

Comments

@Zork33
Copy link
Collaborator

Zork33 commented Jan 23, 2024

Implement BETA version of query service API
@Zork33 Zork33 added the enhancement New feature or request label Jan 23, 2024
@Zork33 Zork33 self-assigned this Jan 23, 2024
@Zork33
Copy link
Collaborator Author

Zork33 commented Jan 23, 2024

Closes #343

This was referenced Jan 24, 2024
Closed
Closed
Closed
@Zork33 Zork33 changed the title dev: Make query service API dev: Add Query service to the SDK Jan 24, 2024
@Zork33 Zork33 changed the title dev: Add Query service to the SDK Add Query service to the SDK Jan 24, 2024
@Zork33 Zork33 mentioned this issue Jan 24, 2024
Closed
@Zork33
Copy link
Collaborator Author

Zork33 commented Jan 24, 2024

Now for each Table, Query client we create its own instances of discovery service, session pool etc. It seems logical to use the same instances in both clients. These is not done as part of #342

Also, it is necessary to clarify how topics are related to sessions. Preliminary sends may be bound to transactions. In this case, sessions are obtained from table service.

@Zork33
Copy link
Collaborator Author

Zork33 commented Feb 1, 2024

closes #337

@Zork33 Zork33 closed this as completed May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Zork33 Zork33

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add Query service to the SDK Zork33/ydb-nodejs-sdk
1 participant
@Zork33