@dhermes Can you glance at this one? I lack sufficient context.

@garye I'd guess there would be broken tests, but it's hard to say right now because there is a bad release of protobuf that is breaking everything

@dhermes Update on the protobuf thing: googleapis/api-client-staging#159

I restarted the CircleCI build: https://circleci.com/gh/GoogleCloudPlatform/google-cloud-python/865

Hopefully it works?

Can someone help me finish this off? The test failures don't really seem related. Getting this fix pushed and released will unblock a number of users.

@garye Just re-started the CircleCI build: https://circleci.com/gh/GoogleCloudPlatform/google-cloud-python/876

Hopefully the bad imports are fully resolved. I'll follow-up if not.

@garye So we've got a unit test failing that is actually caused by this PR:

    def test_invalid_empty_second_chunk(self):
        from google.cloud.bigtable.row_data import InvalidChunk
    
        chunks = _generate_cell_chunks(['', ''])
        first = chunks[0]
        first.row_key = b'RK'
        first.family_name.value = 'A'
        first.qualifier.value = b'C'
        response = _ReadRowsResponseV2(chunks)
        iterator = _MockCancellableIterator(response)
        prd = self._make_one(iterator)
        with self.assertRaises(InvalidChunk):
>           prd.consume_next()
E           AssertionError: InvalidChunk not raised

ISTM the right move is just to delete this test case since it no longer applies?

yeah, i think that's an invalid test case

Ping me when this PR is updated with the test case removed? You can ignore the speech test failures. It looks like the underlying protobuf libraries are (now correctly) using single instead of double precision for message fields with single precision

Yeah, thanks I noticed that and (hopefully) pushed a fix.

@garye Ugh, you need to be able to run tox -e lint locally:

bigtable/unit_tests/test_row_data.py:419:1: W293 blank line contains whitespace

I think it should just work to run tox -e pylint. You probably want to export the following environment variables so it only runs on your changes:

$ export GOOGLE_CLOUD_TESTING_REMOTE=upstream  # the name of the remote for this repo
$ export GOOGLE_CLOUD_TESTING_BRANCH=master
$ tox -e lint

LGTM

Thanks guys!

@garye We hope to cut a release in the next 2-3 days as well