Skip to content

fix(framework): fix finished_record count in _devlake_subtasks#8054

Merged
abeizn merged 2 commits into
mainfrom
dev-9
Sep 14, 2024
Merged

fix(framework): fix finished_record count in _devlake_subtasks#8054
abeizn merged 2 commits into
mainfrom
dev-9

Conversation

@d4x1
Copy link
Copy Markdown
Contributor

@d4x1 d4x1 commented Sep 14, 2024 •
edited
Loading

⚠️ Pre Checklist

Please complete ALL items in this checklist, and remove before submitting

  • I have read through the Contributing Documentation.
  • I have added relevant tests.
  • I have added relevant documentation.
  • I will add labels to the PR, such as pr-type/bug-fix, pr-type/feature-development, etc.

Summary

What does this PR do?
Fix finished_record count in _devlake_subtasks.

Does this close any open issues?

Related #7959.

Screenshots

Include any relevant screenshots here.
image

Other Information

Any other information that is important to this PR.

@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Sep 14, 2024
@dosubot dosubot Bot added component/framework This issue or PR relates to the framework pr-type/bug-fix This PR fixes a bug severity/p1 This bug affects functionality or significantly affect ux labels Sep 14, 2024
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Sep 14, 2024
@abeizn abeizn merged commit f0ad6f5 into main Sep 14, 2024
@abeizn abeizn deleted the dev-9 branch September 14, 2024 07:32
narrowizard pushed a commit to narrowizard/incubator-devlake that referenced this pull request Oct 23, 2024
@d4x1 @narrowizard
fix(framework): fix finished_record count in _devlake_subtasks (apach…
269e306 
…e#8054)
@narrowizard narrowizard mentioned this pull request Oct 23, 2024
Merged
klesh pushed a commit that referenced this pull request Oct 23, 2024
@narrowizard @d4x1
Cherry pick #8152 to release v1.0 (#8153)
48dd544 
* fix(framework): fix finished_record count in _devlake_subtasks (#8054)

* feat: not update sub task progress if progress less than 1 pct (#8152)

[Refactor][core]Data inflation when using postgres #8142

---------

Co-authored-by: Lynwee <1507509064@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@abeizn abeizn abeizn approved these changes

Assignees

No one assigned

Labels

component/framework This issue or PR relates to the framework lgtm This PR has been approved by a maintainer pr-type/bug-fix This PR fixes a bug severity/p1 This bug affects functionality or significantly affect ux size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@d4x1 @abeizn