[GitHub] [incubator-devlake] abeizn opened a new pull request, #4267: fix: fetch project name from the projects and add distinct for some d…

["abeizn (via GitHub)" <gi...@apache.org>]

abeizn opened a new pull request, #4267:
URL: https://github.com/apache/incubator-devlake/pull/4267

   ### Summary
   fix: fetch project name from the projects and add distinct for some duplicate data
   
   ### Does this close any open issues?
   Closes #4262 
   
   ### Screenshots
   previous:
   1. dora:
   ![image](https://user-images.githubusercontent.com/101256042/215405576-cbc9df86-7e7e-419c-8baa-58f87a9a1102.png)
   2. engineering-throughput-and-cycle-time:
   ![image](https://user-images.githubusercontent.com/101256042/215405746-1ce45554-4731-4f7c-ad58-72b31f61d6d1.png)
   
   now:
   1. dora:
   ![image](https://user-images.githubusercontent.com/101256042/215405858-051fbc1f-8b3f-4138-867e-33055f4f0fa7.png)
   2. engineering-throughput-and-cycle-time:
   ![image](https://user-images.githubusercontent.com/101256042/215405968-2c93e757-3566-45bf-bb95-e92ce020127c.png)
   
   
   
   
   ### Other Information
   Any other information that is important to this PR.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-devlake] Startrekzky merged pull request #4267: fix: fetch project name from the projects and add distinct for some d…

["Startrekzky (via GitHub)" <gi...@apache.org>]

Startrekzky merged PR #4267:
URL: https://github.com/apache/incubator-devlake/pull/4267


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-devlake] Startrekzky commented on a diff in pull request #4267: fix: fetch project name from the projects and add distinct for some d…

["Startrekzky (via GitHub)" <gi...@apache.org>]

Startrekzky commented on code in PR #4267:
URL: https://github.com/apache/incubator-devlake/pull/4267#discussion_r1090411725


##########
grafana/dashboards/EngineeringThroughputAndCycleTime.json:
##########
@@ -167,7 +167,7 @@
           "hide": false,
           "metricColumn": "none",
           "rawQuery": true,
-          "rawSql": "SELECT\n  DATE_ADD(date(pr.created_date), INTERVAL -$interval(date(pr.created_date))+1 DAY) as time,\n  count(id) as \"PR: Opened\",\n  count(distinct case when pr.merged_date is not null then id else null end) as \"PR: Merged\"\nFROM pull_requests pr\n  join project_mapping pm on pr.base_repo_id = pm.row_id\nWHERE\n  $__timeFilter(pr.created_date)\n  and pm.project_name in ($project)\ngroup by 1\n",
+          "rawSql": "SELECT\n  DATE_ADD(date(pr.created_date), INTERVAL -$interval(date(pr.created_date))+1 DAY) as time,\n  count(distinct id) as \"PR: Opened\",\n  count(distinct case when pr.merged_date is not null then id else null end) as \"PR: Merged\"\nFROM pull_requests pr\n  join project_mapping pm on pr.base_repo_id = pm.row_id\nWHERE\n  $__timeFilter(pr.created_date)\n  and pm.project_name in ($project)\ngroup by 1\n",

Review Comment:
   Let's use pr.id to make it preciser?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org