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

perf: use jobs.getQueryResults to download result sets #347

Closed

Commits on Oct 29, 2020

  1. perf: use jobs.getQueryResults to download result sets

    Since `getQueryResults` was already used to wait for the job to finish,
    this avoids an additional call to `tabledata.list`. The first page of
    results are cached in-memory.
    
    Additional changes will come in the future to avoid calling the BQ
    Storage API when the cached results contain the full result set.
    tswast committed Oct 29, 2020
    Configuration menu
    Copy the full SHA
    983c8d2 View commit details
    Browse the repository at this point in the history

Commits on Oct 30, 2020

  1. fix: validate the query results cache before using

    Also, move to thread-local variables for values that were
    intended to track parameters across methods.
    tswast committed Oct 30, 2020
    Configuration menu
    Copy the full SHA
    f52ed71 View commit details
    Browse the repository at this point in the history

Commits on Nov 2, 2020

  1. Configuration menu
    Copy the full SHA
    e149360 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    9b5920f View commit details
    Browse the repository at this point in the history
  3. fix dbapi tests

    tswast committed Nov 2, 2020
    Configuration menu
    Copy the full SHA
    07e6043 View commit details
    Browse the repository at this point in the history
  4. fix system test

    startIndex is no longer passed to the iterator
    It is used in the initial (cached) call to
    getQueryResults
    tswast committed Nov 2, 2020
    Configuration menu
    Copy the full SHA
    af2e2cc View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    540d530 View commit details
    Browse the repository at this point in the history
  6. blacken

    tswast committed Nov 2, 2020
    Configuration menu
    Copy the full SHA
    6e83fbf View commit details
    Browse the repository at this point in the history