Skip to content
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

Adding Bigtable row filters for boolean flags. #1307

Merged
merged 1 commit into from
Dec 19, 2015

Conversation

dhermes
Copy link
Contributor

@dhermes dhermes commented Dec 18, 2015

No description provided.

@dhermes dhermes added the api: bigtable Issues related to the Bigtable API. label Dec 18, 2015
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 18, 2015
@@ -253,3 +324,21 @@ def to_pb(self):
:returns: The converted current object.
"""
return data_pb2.RowFilter(cells_per_column_limit_filter=self.num_cells)


class StripValueTransformerFilter(_BoolFilter):

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@tseaver
Copy link
Contributor

tseaver commented Dec 18, 2015

LGTM except for my ordering question.

@dhermes
Copy link
Contributor Author

dhermes commented Dec 19, 2015

@tseaver Will merge on Travis success

@dhermes
Copy link
Contributor Author

dhermes commented Dec 19, 2015

For some reason Travis didn't trigger.

dhermes added a commit that referenced this pull request Dec 19, 2015
Adding Bigtable row filters for boolean flags.
@dhermes dhermes merged commit 4861e7e into googleapis:master Dec 19, 2015
@dhermes dhermes deleted the bigtable-row-filter-4 branch December 19, 2015 00:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: bigtable Issues related to the Bigtable API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants