Skip to content

fix: allow colon in column qualifier #297

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.

Sign up for GitHub

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

Jump to bottom
Open
jaeyeolshin wants to merge 2 commits into
base: main
Choose a base branch
from
Open

fix: allow colon in column qualifier #297

jaeyeolshin wants to merge 2 commits into from

Conversation

@jaeyeolshin
Copy link

@jaeyeolshin jaeyeolshin commented Jan 2, 2026 •
edited
Loading

This PR aims to fix #296.

Additionally this changes the behavior read or lookup commands when ":" is passed as the columns argument. Currently when ":" is passed, cbt reads rows with an empty family and this causes the following server error.

2026/01/02 20:55:08 Reading row: rpc error: code = InvalidArgument desc = Error in field 'family_name_regex_filter' : argument must not be empty

But this PR changes it to read rows with an empty column qualifier, which is allowed to bigtable.

@jaeyeolshin jaeyeolshin requested review from a team and igorbernstein2 as code owners January 2, 2026 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@igorbernstein2 igorbernstein2 Awaiting requested review from igorbernstein2 igorbernstein2 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Colon is not allowed in column qualifiers when specifying columns argument

2 participants

@jaeyeolshin @jaeyeol-moloco