Skip to content

Conversation

@MeredithAnya
Copy link
Member

@MeredithAnya MeredithAnya commented Feb 2, 2026

Main change in this PR is to remove target_host from the copy tables workflow. Instead, you just pick the source host you want to run the create table on and it will run it for the whole cluster. In addition to that there are some other UI/Backend changes that were added for verification so that you can easily tell if the command created all the tables on all the hosts (including the new replica/shard)

UI Changes

  • Summary line: "X of Y hosts verified" uses the count of hosts
  • Incomplete hosts: Lists the hosts with missing tables and which tables they are missing

Backend Changes

  • verify_tables_on_replicas checks that all the tables were created on the tables, and if they weren't returns those specific hosts and which tables they are missing
  • Added is_single_node mainly for local dev and testing purposes

🤖 Generated with Claude Code

Updates the copy tables admin tool with improved UX and verification:
- Remove target host selection (executes on cluster via ON CLUSTER)
- Add table verification section showing verified vs incomplete hosts
- Display only incomplete hosts with missing tables
- Consolidate styling with base styles and theme colors
- Improve layout with cleaner sections and consistent spacing
- Update backend to return verification status (verified count and incomplete hosts)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@MeredithAnya MeredithAnya requested a review from a team as a code owner February 2, 2026 17:51
@MeredithAnya MeredithAnya changed the title feat(admin): Improve copy tables UI and add host verification feat(admin): remove target_host, add host verification, ui changes Feb 2, 2026
@MeredithAnya
Copy link
Member Author

MeredithAnya commented Feb 2, 2026

SUCCESS

Screenshot 2026-02-02 at 9 31 23 AM

FAILURE

Screenshot 2026-02-02 at 9 18 01 AM

note: "no cluster" is really only possible with local dev since if you try to run ON CLUSTER without zk you get an exception

@MeredithAnya MeredithAnya enabled auto-merge (squash) February 2, 2026 18:43
@MeredithAnya MeredithAnya merged commit 0180c1b into master Feb 2, 2026
33 checks passed
@MeredithAnya MeredithAnya deleted the meredith/1-30-26-copy-tables branch February 2, 2026 19:15
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