Skip to content
This repository has been archived by the owner on Jun 14, 2024. It is now read-only.

[WIP] Refactor Hyperspace Index Configs to allow support for addition of new index #357

Closed
wants to merge 5 commits into from

Conversation

thugsatbay
Copy link
Contributor

@thugsatbay thugsatbay commented Feb 12, 2021

What is the context for this pull request?

Refactor Hyperspace Index Configs to allow support for addition of new index. Like Bloom Filter or Partition Elimination Index.

  • Tracking Issue: #341
  • Parent Issue: NA
  • Dependencies: NA

What changes were proposed in this pull request?

  1. Separating Case and Builder classes of configs
  2. Putting al builder in designated package
  3. Having case classes in same class name as package name
  4. Add a function in config that displays all index available in hyperspace

Does this PR introduce any user-facing change?

Yes, explained in proposed changes.

How was this patch tested?

TODO

@thugsatbay thugsatbay mentioned this pull request Feb 18, 2021
12 tasks
@clee704
Copy link

clee704 commented Jun 22, 2021

Cleaning up old PRs - This PR is superseded by #443.

@clee704 clee704 closed this Jun 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants