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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

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

Merged
merged 7 commits into from Nov 4, 2020
Merged

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

merged 7 commits into from Nov 4, 2020

Commits on Nov 3, 2020

  1. refactor: break job into multiple modules 

    Original paths are retained for backwards compatibility.
    @tswast
    tswast committed Nov 3, 2020
    Copy the full SHA
    ce147ef View commit details
    Browse the repository at this point in the history

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

    Replaces `tabledata.list` when `RowIterator` is used for query results.
This likely also fixes a few edge cases around BigQuery scripting jobs.
    @tswast
    tswast committed Nov 3, 2020
    Copy the full SHA
    1957a7d View commit details
    Browse the repository at this point in the history

Commits on Nov 4, 2020

  1. Merge branch 'master' into issue362-getQueryResults-RowIterator

    @tswast
    tswast committed Nov 4, 2020
    Copy the full SHA
    ea97892 View commit details
    Browse the repository at this point in the history

  2. revert unnecessary changes to _get_query_results

    @tswast
    tswast committed Nov 4, 2020
    Copy the full SHA
    9e88464 View commit details
    Browse the repository at this point in the history

  3. simplify RowIterator. no need to hack Table object

    @tswast
    tswast committed Nov 4, 2020
    Copy the full SHA
    8702674 View commit details
    Browse the repository at this point in the history

  4. fix tests for bqstorage warning

    @tswast
    tswast committed Nov 4, 2020
    Copy the full SHA
    5df559e View commit details
    Browse the repository at this point in the history

  5. populate location

    @tswast
    tswast committed Nov 4, 2020
    Copy the full SHA
    daed21e View commit details
    Browse the repository at this point in the history