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.

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

fix: allow partial streaming_buffer statistics #37

Merged
merged 6 commits into from Feb 14, 2020
Merged

fix: allow partial streaming_buffer statistics #37

merged 6 commits into from Feb 14, 2020

Commits on Feb 13, 2020

  1. fix: allow partial streaming_buffer statistics. 

    Previously, the BQ backend would supply all fields of the
streamingBuffer statistics or none.  This is no longer the
case, so we relax construction to not depend on all values
being present.

Related: internal issue b/148720220
    @shollyman
    shollyman committed Feb 13, 2020
    Copy the full SHA
    236db9a View commit details
    Browse the repository at this point in the history

  2. lint / blacken

    @shollyman
    shollyman committed Feb 13, 2020
    Copy the full SHA
    320aee3 View commit details
    Browse the repository at this point in the history

  3. change checks to avoid KeyError

    @shollyman
    shollyman committed Feb 13, 2020
    Copy the full SHA
    e17f828 View commit details
    Browse the repository at this point in the history

Commits on Feb 14, 2020

  1. Fallback to None rather than a missing attribute

    @shollyman
    shollyman committed Feb 14, 2020
    Copy the full SHA
    71270ef View commit details
    Browse the repository at this point in the history

  2. add another test variant

    @shollyman
    shollyman committed Feb 14, 2020
    Copy the full SHA
    04b2bee View commit details
    Browse the repository at this point in the history

  3. assertion

    @shollyman
    shollyman committed Feb 14, 2020
    Copy the full SHA
    6d1ad30 View commit details
    Browse the repository at this point in the history