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

Making common subclass for Bigtable DirectRow and ConditionalRow. #1594

Merged
merged 1 commit into from
Mar 11, 2016

Conversation

dhermes
Copy link
Contributor

@dhermes dhermes commented Mar 10, 2016

This is a follow-up to #1557 and prevents strange-ly true statements like isinstance(cond_row, DirectRow).

Sorry it took me so many days. Code has been completely written for 2+ days but I haven't ever had my laptop near internet long enough to push it.

@dhermes dhermes added the api: bigtable Issues related to the Bigtable API. label Mar 10, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 10, 2016
@@ -63,5 +63,6 @@ level. For example:
:members:
:undoc-members:
:show-inheritance:
:inherited-members:

This comment was marked as spam.

This comment was marked as spam.

@tseaver
Copy link
Contributor

tseaver commented Mar 10, 2016

LGTM

This is a follow-up to googleapis#1557 and prevents strange-ly true
statements like `isinstance(cond_row, DirectRow)`.
@dhermes
Copy link
Contributor Author

dhermes commented Mar 11, 2016

@tseaver I rebased after #1602. Will merge once Travis passes.

dhermes added a commit that referenced this pull request Mar 11, 2016
Making common subclass for Bigtable DirectRow and ConditionalRow.
@dhermes dhermes merged commit 97eb7f5 into googleapis:master Mar 11, 2016
@dhermes dhermes deleted the pr-1557-followup branch March 11, 2016 20:42
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