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

PXD-2916: add unique indexes for uniqueKeys #12

Merged
merged 4 commits into from
Apr 18, 2019
Merged

Conversation

fantix
Copy link

@fantix fantix commented Feb 14, 2019

Auto migration is in uc-cdis/datamodelutils#6.

This PR adds unique indexes for:

  • (project_id, submitter_id) of other nodes

And indexes on these columns are replaced with unique ones:

  • program.name
  • project.code

Improvements

  • added unique indexes for uniqueKeys

return tuple(key_indexes) + tuple(lower_key_indexes)
rv = tuple(key_indexes) + tuple(lower_key_indexes)

if secondary_keys:
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, this is wrong - it's a flatten 1-dimension array, not the original 2-dimension array.

@fantix
Copy link
Author

fantix commented Feb 22, 2019

Closing in favor of #14, the unique check will be done in extra queries.

@fantix fantix closed this Feb 22, 2019
@fantix fantix deleted the feat/uniqueKeys branch February 22, 2019 23:45
@fantix fantix restored the feat/uniqueKeys branch April 17, 2019 16:05
@fantix fantix reopened this Apr 17, 2019
This partially reverts commit d32ea0a.
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.

2 participants