Skip to content

Conversation

alexr-bq
Copy link
Collaborator

@alexr-bq alexr-bq commented Oct 2, 2025

  • Adds support for database ID with cluster client
  • Moves Move/Copy commands from standalone to shared client

@alexr-bq alexr-bq requested a review from jbrinkman October 2, 2025 16:00
@alexr-bq alexr-bq requested a review from a team as a code owner October 2, 2025 16:00
Signed-off-by: Alex Rehnby-Martin <[email protected]>
Signed-off-by: Alex Rehnby-Martin <[email protected]>
Signed-off-by: Alex Rehnby-Martin <[email protected]>
Signed-off-by: Alex Rehnby-Martin <[email protected]>
Signed-off-by: Alex Rehnby-Martin <[email protected]>
@yipin-chen
Copy link
Collaborator

@alexr-bq You also need to add the support for select command. Please review issue 4810. Thanks

Signed-off-by: Alex Rehnby-Martin <[email protected]>
Signed-off-by: Alex Rehnby-Martin <[email protected]>
Copy link

@affonsov affonsov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's missing the select command, which is basically reverting this pr for the select #70. The only difference is that select should only be available for standalone batch and not cluster batch

Signed-off-by: Alex Rehnby-Martin <[email protected]>
@alexr-bq
Copy link
Collaborator Author

alexr-bq commented Oct 5, 2025

It's missing the select command, which is basically reverting this pr for the select #70. The only difference is that select should only be available for standalone batch and not cluster batch

It seems like Select functionality is already in. Pushed a commit that removes it from cluster batch.

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.

3 participants