Skip to content

Conversation

@wangbo
Copy link
Contributor

@wangbo wangbo commented Mar 1, 2024

Proposed changes

Routine load support workload group
1 user can specify a workload group for routine load.
2 if user does not specify a workload group, then we could get workload group from user's property.
3 if user's propery does not have a valid workload group, then it will throw an exception.

@doris-robot
Copy link

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

@wangbo
Copy link
Contributor Author

wangbo commented Mar 1, 2024

run compile

@wangbo
Copy link
Contributor Author

wangbo commented Mar 1, 2024

run p0

@wangbo wangbo force-pushed the 0301_routine_load_support_wg branch from b63ed84 to f1d8ddd Compare March 4, 2024 04:18
@wangbo
Copy link
Contributor Author

wangbo commented Mar 4, 2024

run compile

@wangbo
Copy link
Contributor Author

wangbo commented Mar 4, 2024

run buildall

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Mar 8, 2024
@github-actions
Copy link
Contributor

github-actions bot commented Mar 8, 2024

PR approved by at least one committer and no changes requested.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 8, 2024

PR approved by anyone and no changes requested.

Copy link
Contributor

@sollhui sollhui left a comment

Choose a reason for hiding this comment

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

please add doc to introduce how to use workload_group in routine load.

Copy link
Contributor

@sollhui sollhui left a comment

Choose a reason for hiding this comment

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

LGTM

@yiguolei yiguolei merged commit 5d4a949 into apache:master Mar 11, 2024
@yiguolei yiguolei mentioned this pull request Mar 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. reviewed workload-group

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants