+
Skip to content

[Bug] int_github__pull_request_times.sql is dropping rows where no reviewers are explicitly requested #56

@samkessaram

Description

@samkessaram

Is there an existing issue for this?

  • I have searched the existing issues

Describe the issue

The spine of the int_github__pull_request_times model is first_request_time, which is a CTE that inner joins pull requests to requested_reviewer_history. However, people can review pull requests without being explicitly requested, and then those pull requests can be merged. But because of the inner join, PRs without requested reviewers are dropped. This means that downstream, some merged PRs have a null merged_at timestamp.

Relevant error log or model output

No response

Expected behavior

Merged PRs should have a non-null merged_at timestamp.

Possible solution

Change the inner join to be a left join.

dbt Project configurations

n/a

Package versions

  - package: fivetran/github
    version: 0.7.0

What database are you using dbt with?

snowflake

How are you running this dbt package?

dbt Core™

dbt Version

1.8.0

Additional Context

No response

Are you willing to open a PR to help address this issue?

  • Yes.
  • Yes, but I will need assistance.
  • No.

Metadata

Metadata

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions

    点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载