Skip to content

Extend Nandi with yugabyte-specific instructions - #159

Draft
BBrannick wants to merge 20 commits into
masterfrom
yb-add-index
Draft

Extend Nandi with yugabyte-specific instructions#159
BBrannick wants to merge 20 commits into
masterfrom
yb-add-index

Conversation

@BBrannick

@BBrannick BBrannick commented Aug 11, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

To support GC working with Yugabyte databases, add support to Nandi for generating migrations with yugabyte-specific syntax.

  • Separates renderers and migrations into Abstract classes, with implementations for Postgres and Yugabyte.
  • Yugabyte logic is encapsulated in a lib/nandi/yugabyte directory for separation/easier extraction.
  • Pulls instruction->validator mapping into the instruction class itself for better extensibility.
  • Adds yugabyte-specific add_index command to allow bucketing.

@BBrannick
BBrannick force-pushed the yb-add-index branch 2 times, most recently from e1a0f23 to 7f499a3 Compare August 14, 2026 12:12
BBrannick and others added 18 commits August 24, 2026 17:18
@BBrannick
Add add_index_yb instruction for yb-specific indexes
803010f
@BBrannick
Add design doc for add_index_yb yugabyte_database validation
f3fb305 
Specs out fixing the missing yugabyte_database config attribute and
adding validation so add_index_yb can only be used when the target
database is configured as YugabyteDB.
@BBrannick @claude
Add yugabyte_database config attribute to Database
5e190af 
Add yugabyte_database as a configuration attribute to MultiDatabase::Database
to support YugaByte-specific database validation. The attribute defaults to
false when not configured and respects explicit true/false values. The existing
Config#yugabyte_database? method now correctly delegates to this attribute.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@BBrannick
Validate add_index_yb is only used on yugabyte-configured databases
c504618
@BBrannick
Refactor migration and renderer into db-specific implementations
8287461
@BBrannick
Add database-specific renderer config
3f651ac
@BBrannick
Remove superpowers doc
9a2437b
@BBrannick
Add DB config to instruction validators
ca5dafa
@BBrannick
Remove yugabyte-database validation check
a4e27b2
@BBrannick
Move instruction->validator mapping to instruction class
b4fd126
@BBrannick
Minor linting fixes
358203c
@BBrannick
Fix spec issue with nil db_name
f652d94
@BBrannick
Move yugabyte extension to dedicated directory
a20a576
@BBrannick
Create nandi dir in yugabyte dir for correct pathing
4554a39
@BBrannick
Require yugabyte migration class from compile generator
b0199c4
@BBrannick
Use db-specific superclass when generating migration
b8017b7
@BBrannick
Use db-specific superclass name in migration generators
b288665
@BBrannick
Extract database-specific constants
9c00ab4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@BBrannick