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

Add query_id of last executed statement to Adapter Response #891

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Add query_id of last executed statement to Adapter Response #891

wants to merge 2 commits into from

Commits on Jul 31, 2024

  1. Add query_id of last executed statement to Adapter Response 

    I want to be able to capture the `query_id` of the statement used to load a model.

I ripped this off of the dbt-snowflake adapter and how they are setting `query_id` on the adapter response:

https://github.com/dbt-labs/dbt-snowflake/blob/f95b9192f6eec9af4e30eaab87f9e3412febf7d1/dbt/adapters/snowflake/connections.py#L456-L461

I'm sure this doesn't handle 100% of accurately capturing the correct query ID for a particular load (I'm sure there are cases where multiple statements are run in order to perform the model or snapshot load), but I think this is a good start.
    @KeltonKarboviak
    KeltonKarboviak authored Jul 31, 2024
    Configuration menu
    Copy the full SHA
    e45994c View commit details
    Browse the repository at this point in the history

  2. Add default of -1 for query_id

    @KeltonKarboviak
    KeltonKarboviak committed Jul 31, 2024
    Configuration menu
    Copy the full SHA
    2c52d79 View commit details
    Browse the repository at this point in the history