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

feat: expose the query source of a rowiterator via SourceJob() #4748

Merged
merged 3 commits into from Sep 15, 2021
Merged

feat: expose the query source of a rowiterator via SourceJob() #4748

merged 3 commits into from Sep 15, 2021

Commits on Sep 13, 2021

  1. feat: expose the query source of a rowiterator via SourceJob() 

    RowIterator can currently come from calling Read() on a Table,
Read() on a Job, and Read() directly from a Query.  The third invocation
is used for fast path queries and doesn't return a query identifier
suitable for gathering statistics.

This PR exposes a SourceJob() method on RowIterator to address this
issue.  Users still get the benefits of optimized query execution, but
can get the reference to a Job for looking up additional statistics
should they need it.
    @shollyman
    shollyman committed Sep 13, 2021
    Copy the full SHA
    7afce4c View commit details
    Browse the repository at this point in the history

Commits on Sep 15, 2021

  1. Merge branch 'master' into fr-rowiterator-source

    @shollyman
    shollyman committed Sep 15, 2021
    Copy the full SHA
    5fed42f View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' into fr-rowiterator-source

    @shollyman
    shollyman committed Sep 15, 2021
    Copy the full SHA
    1daec08 View commit details
    Browse the repository at this point in the history