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

Refactor index checks #2541

Merged
merged 6 commits into from
Oct 17, 2023
Merged

Refactor index checks #2541

merged 6 commits into from
Oct 17, 2023

Conversation

casey
Copy link
Collaborator

@casey casey commented Oct 14, 2023

  • Remove index checks in server, just return empty responses.
  • Create all tables when index is created, so all index functions can be called, and will just return empty responses when the index doesn't exist.
  • Add explicit checks to those command which don't make sense without an index.

This just cleans up the code a bunch.

@casey casey force-pushed the refactor-index-checks branch from bfd62cd to f7a497e Compare October 14, 2023 22:38
@casey casey enabled auto-merge (squash) October 17, 2023 20:16
@casey casey merged commit fadeda8 into ordinals:master Oct 17, 2023
6 checks passed
@casey casey deleted the refactor-index-checks branch October 17, 2023 20:22
@casey casey restored the refactor-index-checks branch October 17, 2023 20:22
@casey casey deleted the refactor-index-checks branch October 17, 2023 20:22
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.

Don't use existence of table to determine whether rune index exists
2 participants