Skip to content

fix: outdated SQL queries due to PR status changes#4548

Merged
abeizn merged 4 commits into
apache:mainfrom
merico-ai:fix#4524
Mar 1, 2023
Merged

fix: outdated SQL queries due to PR status changes#4548
abeizn merged 4 commits into
apache:mainfrom
merico-ai:fix#4524

Conversation

@abeizn
Copy link
Copy Markdown
Contributor

@abeizn abeizn commented Feb 28, 2023

Summary

qutdated SQL queries due to PR status changes

  1. update github pr extractor
  2. update some dashboards by using pr status

Does this close any open issues?

Closes #4524

Screenshots

user---- engineering throughput and cycle time:
image
image

user---- engineering overview:
image

fixed---- engineering throughput and cycle time:
image
image

fixed---- engineering overview:
image

Other Information

Any other information that is important to this PR.

@abeizn abeizn changed the title fix: qutdated SQL queries due to PR status changes fix: outdated SQL queries due to PR status changes Feb 28, 2023
Copy link
Copy Markdown
Contributor

@Startrekzky Startrekzky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@abeizn
Copy link
Copy Markdown
Contributor Author

abeizn commented Mar 1, 2023

lint failed by: #4521 not by this

@abeizn abeizn merged commit 78bf4f9 into apache:main Mar 1, 2023
abeizn added a commit that referenced this pull request Mar 1, 2023
* fix: qutdated SQL queries due to PR status changes

* fix: github pr e2e failed

* fix: update dashboard

* fix: update dashboard
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug][GitHub dashboard] Outdated SQL queries due to PR status changes

2 participants