[Bug] requested_reviewers is null for pull requests that have no reviews yet
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 35/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 停滞
- 技術スタック
- sql
調査の方向性
Read the int_github__pull_request_reviewers model, especially the joined CTE, and trace how int_github__issue_joined uses pull_request_id to populate github__pull_requests. Reproduce the five seeded cases with open review requests and no submitted reviews, then confirm the expected count changes from 34 to 39 populated requested_reviewers; pull request #87 is already open for this issue.
索引モデルが issue の本文から書いたものです。
説明
Is there an existing issue for this?
- I have searched the existing issues
Describe the issue
int_github__pull_request_reviewers full outer joins actual_reviewers to requested_reviewers, but selects source_relation and pull_request_id only from actual_reviewers. For a pull request that has requested reviewers but no submitted reviews yet, the row comes from requested_reviewers alone, so pull_request_id and source_relation are null. int_github__issue_joined joins on pull_request_id, so that row never matches and requested_reviewers is null in github__pull_requests. These are the pull requests still waiting on review, which is where the column is most useful.
Relevant error log or model output
With the integration test seeds, 5 pull requests have an open review request and no reviews (for example pull_request_id 408105961, 407401898, 516351583, 521867065, 442423917). All 5 have requested_reviewers = null in github__pull_requests, even though the requested user exists in user. Overall 34 of 167 pull requests have requested_reviewers populated.
Expected behavior
requested_reviewers lists the requested reviewers whether or not anyone has reviewed yet. With the seeds, 39 of 167 pull requests.
Possible solution
Coalesce source_relation and pull_request_id from both sides of the full outer join in the joined CTE. I have a PR ready.
dbt Project configurations
Package integration_tests project.
Package versions
dbt_github 1.5.0 (main)
What database are you using dbt with?
duckdb
How are you running this dbt package?
dbt Core
dbt Version
1.12.6
Are you willing to open a PR to help address this issue?
- Yes.
- 主要言語
- 言語のデータがありません
- スター
- 21
- フォーク
- 25
- 平均マージ
- 1日 41分
- マージ済み PR（30日）
- 2
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
fivetran/dbt_github のほかの issue
-
next-release
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
fivetran/dbt_github#74 ·
-
good first issue type:enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
fivetran/dbt_github#61 ·
-
type:enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
fivetran/dbt_github#34 · コメント 1 件 ·
-
type:enhancement
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
fivetran/dbt_github#69 · リアクション 1 件 ·
-
Add Team supportオープンtype:enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 30/100
fivetran/dbt_github#70 · コメント 5 件 ·
fivetran/dbt_github の issue をすべて見る
似ている issue
-
ODS datetime imports reject fractional seconds and exports drop them対応中かも @IlyaKhalafi が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 71/100
Chain-Love/chain-love#5295 ·
メンテナーはふだん 5 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
fivetran/dbt_recurly#55 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
aws-powertools/powertools-lambda-python#8800 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
TauricResearch/TradingAgents#1693 ·
メンテナーはふだん 2 日以内に返信