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

[yugabyted] Use pgxpool instead of pgx for yugabyted-ui api server #15237

Closed
djiang9001 opened this issue Dec 6, 2022 · 1 comment
Closed

[yugabyted] Use pgxpool instead of pgx for yugabyted-ui api server #15237

djiang9001 opened this issue Dec 6, 2022 · 1 comment
Assignees
@djiang9001
Labels
area/db-usability issue related to DB usability project. Including yugabyted cli and yugabyted ui issues.

Comments

@djiang9001
Copy link
Contributor

Description

We need to switch pgx to pgxpool for the yugabyted-ui api server since pgx is not concurrency safe, which will cause certain functionalities on the yugabyted ui to fail.
@djiang9001 djiang9001 added the area/db-usability issue related to DB usability project. Including yugabyted cli and yugabyted ui issues. label Dec 6, 2022
@djiang9001 djiang9001 self-assigned this Dec 6, 2022
djiang9001 added a commit that referenced this issue Dec 6, 2022
@djiang9001
[#15237][yugabyted-ui] Use pgxpool instead of pgx for yugabyted-ui ap…
2a938d6 
…i server

Summary: Use pgxpool instead of pgx for yugabyted-ui api server

Test Plan: no test plan

Reviewers: nikhil

Reviewed By: nikhil

Subscribers: djiang

Differential Revision: https://phabricator.dev.yugabyte.com/D21567
jayant07-yb pushed a commit to jayant07-yb/yugabyte-db that referenced this issue Dec 7, 2022
@djiang9001
[yugabyte#15237][yugabyted-ui] Use pgxpool instead of pgx for yugabyt…
45a23ed 
…ed-ui api server

Summary: Use pgxpool instead of pgx for yugabyted-ui api server

Test Plan: no test plan

Reviewers: nikhil

Reviewed By: nikhil

Subscribers: djiang

Differential Revision: https://phabricator.dev.yugabyte.com/D21567
@djiang9001
Copy link
Contributor Author

Code changes landed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@djiang9001 djiang9001

Labels
area/db-usability issue related to DB usability project. Including yugabyted cli and yugabyted ui issues.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@djiang9001