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

ysql_dump: create index before data loading #6653

Open
jaki opened this issue Dec 14, 2020 · 1 comment
Open

ysql_dump: create index before data loading #6653

jaki opened this issue Dec 14, 2020 · 1 comment
Assignees
@m-iancu
Labels
area/ysql Yugabyte SQL (YSQL) kind/enhancement This is an enhancement of an existing feature priority/medium Medium priority issue

Comments

@jaki
Copy link
Contributor

jaki commented Dec 14, 2020 •
edited by yugabyte-ci
Loading

Jira Link: DB-1607
ysql_dump creates index after data loading. Look into why that choice is made in pg_dump (see #1104 (comment)). If there is no strong reason, we should move index creation before data loading. This is because backfilling an index is expensive and not fully supported at the moment, so we should avoid it if possible.
@jaki jaki added kind/enhancement This is an enhancement of an existing feature area/ysql Yugabyte SQL (YSQL) area/tools labels Dec 14, 2020
@jaki jaki self-assigned this Dec 14, 2020
@jaki
Copy link
Contributor Author

jaki commented Dec 14, 2020

Workaround is using ysql_dump --schema-only and ysql_dump --data-only.

WesleyW added a commit that referenced this issue Dec 15, 2020
@WesleyW
#6653: Platform backup script should stop prometheus service properly
aba81f6 
Summary: Platform backup currently stops platform.service twice instead of stopping prometheus and platform.

Test Plan: eyed it

Reviewers: sanketh, arnav, daniel

Reviewed By: daniel

Subscribers: jenkins-bot, yugaware

Differential Revision: https://phabricator.dev.yugabyte.com/D10132
WesleyW added a commit that referenced this issue Jan 12, 2021
@WesleyW
#6653: Platform backup script should stop prometheus service properly
e85f92b 
Summary: Platform backup currently stops platform.service twice instead of stopping prometheus and platform.

Test Plan: eyed it

Reviewers: sanketh, arnav, daniel

Reviewed By: daniel

Subscribers: jenkins-bot, yugaware

Differential Revision: https://phabricator.dev.yugabyte.com/D10132
@yugabyte-ci yugabyte-ci added the priority/medium Medium priority issue label Jun 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@m-iancu m-iancu

Labels
area/ysql Yugabyte SQL (YSQL) kind/enhancement This is an enhancement of an existing feature priority/medium Medium priority issue
Projects
YQL-beta
Status: No status
Milestone
No milestone
Development

No branches or pull requests
4 participants
@m-iancu @tverona1 @jaki @yugabyte-ci