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

Add support for standardizable SQL tasks #2

Merged
merged 10 commits into from
Feb 19, 2025
Merged

Add support for standardizable SQL tasks #2

merged 10 commits into from
Feb 19, 2025

Conversation

weaverba137
Copy link
Member

This PR adds support for some standardizable SQL tasks:

  • Create indexes on a column or a group of columns.
  • Create q3c index on columns corresponding to RA, Dec.
  • Create a primary key on a table that doesn't already have one, which is a common occurrence when ingesting a new table.

There are other possible standardizable SQL tasks of course, e.g.:

  • Setting permissions.
  • Creating a schema.
  • Truncating a table to reload it.

The intention of this PR is to create a framework for adding these additional tasks in later PRs.

@jacquesalice if you have suggestions about how to modify the index creation for tablespaces, please comment. I've seen the documentation on Confluence, so I know the syntax, but we might need to compile a list of available tablespaces and determine, do the tablespaces have the same naming convention on all servers?

@weaverba137 weaverba137 self-assigned this Jan 24, 2025
@weaverba137
Copy link
Member Author

@rnikutta, @jacquesalice I have created a new ticket #6 to discuss supporting tablespaces independently of this PR. Therefore I suggest merging this without doing additional work on tablespaces right now.

@weaverba137 weaverba137 merged commit 67a2994 into main Feb 19, 2025
24 checks passed
@weaverba137 weaverba137 deleted the sql-support branch February 19, 2025 21:11
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