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

Try DISTKey migration only if key is specified #212

Merged
merged 1 commit into from
May 2, 2021
Merged

Conversation

alok87
Copy link
Contributor

@alok87 alok87 commented May 2, 2021

Otherwise it can be AUTO or manually maintained by the user

Separate note: #211 DIST key support needs to be added for multi node setups.

After this change, loader will only log the diff:

I0502 10:29:01.294592   12633 redshift.go:1089] customers, SortKey is different, SortKey in config: [], target: [id]
I0502 10:29:01.294602   12633 redshift.go:1096] customers, SortKey is AUTO or manually modified, skipped.
I0502 10:29:01.294610   12633 redshift.go:1115] customers, DistKey is different, DistKey in config: [], target: [user_id]
I0502 10:29:01.294617   12633 redshift.go:1122] customers, DistKey is AUTO or manually modified, skipped.

Otherwise it can be AUTO or manually maintained by the user
@alok87 alok87 changed the title If DIST Key is specified then only try migration Try DISTKey migration only if key is specified May 2, 2021
@alok87 alok87 merged commit 6895ac3 into master May 2, 2021
@alok87 alok87 deleted the dist-key-auto branch May 31, 2021 07:20
alok87 added a commit that referenced this pull request Jun 5, 2021
Try DISTKey migration only if key is specified
alok87 added a commit that referenced this pull request Jun 7, 2021
Try DISTKey migration only if key is specified
alok87 added a commit that referenced this pull request Jun 17, 2021
Try DISTKey migration only if key is specified
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant