Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Enhancement][Log] Add pending to running state change log for task #6831

Merged
merged 2 commits into from
Jun 8, 2022
Merged

[Enhancement][Log] Add pending to running state change log for task #6831

merged 2 commits into from
Jun 8, 2022

Conversation

Astralidea
Copy link
Contributor

What type of PR is this:

  • bug
  • feature
  • enhancement
  • refactor
  • others

Which issues of this PR fixes :

Fixes #6830

Problem Summary(Required) :

The RUNNING state is missing when querying INFORMATION_SCHEMA the execution state of the task through FOLLOWER, so add it.

@wanpengfei-git
Copy link
Collaborator

[FE PR Coverage check]

😞 fail : 2 / 35 (05.71%)

file detail

path covered line new line coverage
🔵 com/starrocks/scheduler/TaskManager.java 0 33 00.00%
🔵 com/starrocks/scheduler/TaskRunManager.java 2 2 100.00%

@chaoyli chaoyli changed the title Add pending to running state change log for task [Enhancement][Log] Add pending to running state change log for task Jun 8, 2022
@chaoyli chaoyli merged commit 8df6596 into StarRocks:main Jun 8, 2022
@Astralidea
Copy link
Contributor Author

@Mergifyio backport branch-2.3

@mergify
Copy link
Contributor

mergify bot commented Jun 8, 2022

backport branch-2.3

✅ Backports have been created

Astralidea pushed a commit that referenced this pull request Jun 8, 2022
…6831) (#6977)

The RUNNING state is missing when querying INFORMATION_SCHEMA the execution state of the task through FOLLOWER, so add it.
abc982627271 pushed a commit to abc982627271/starrocks that referenced this pull request Jun 22, 2022
jaogoy pushed a commit to jaogoy/starrocks that referenced this pull request Nov 15, 2023
… (backport StarRocks#6830) (StarRocks#6831)

* [add command list] (backport StarRocks#6827) (backport StarRocks#6829) (StarRocks#6830)

(cherry picked from commit f9c3449)

Co-authored-by: evelyn.zhaojie <everlyn.zhaojie@gmail.com>
(cherry picked from commit d811b17)

---------

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: evelyn.zhaojie <everlyn.zhaojie@gmail.com>
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.

Add Pending to Running Log in task framework
4 participants