-
Notifications
You must be signed in to change notification settings - Fork 0
Add status command #17
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
Conversation
a36c8ad to
44584ee
Compare
1263f32 to
9e56ae0
Compare
| /** | ||
| * Checks if this table is a partitioned table (relkind = 'p'). | ||
| */ | ||
| async isPartitioned(connection: CommonQueryMethods): Promise<boolean> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Though it adds more code my preference was to delegate the actual logic to Table in order to maintain separation of concerns.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
9e56ae0 to
59ff248
Compare
mthadley
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Left a few comments, but no blockers. Only waiting to approve because I think you'll want to merge in master and reconcile the changes from #16.
Summary
statuscommand that shows the partitioning state of a table (intermediate existence, partition count, mirroring triggers, whether the original is partitioned)--jsonflag for machine-readable outputTest plan
🤖 Generated with Claude Code