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

Add batch source config #496

Merged
merged 4 commits into from
Nov 2, 2022
Merged

Add batch source config #496

merged 4 commits into from
Nov 2, 2022

Conversation

jiangpengcheng
Copy link
Member

Fixes #487

Motivation

Cannot handle BatchSourceConfig

Modifications

Add BatchSourceConfig field

Verifying this change

  • Make sure that the change passes the CI checks.

Documentation

Check the box below.

Need to update docs?

  • doc-required

    (If you need help on updating docs, create a doc issue)

@jiangpengcheng jiangpengcheng requested review from nlu90, freeznet and a team as code owners October 27, 2022 06:31
@github-actions github-actions bot added the doc-required This pr needs a document label Oct 27, 2022
@freeznet freeznet added type/feature Indicates new functionality m/2022-11 labels Oct 31, 2022
nlu90
nlu90 previously approved these changes Oct 31, 2022
Copy link
Contributor

@danpi danpi left a comment

Choose a reason for hiding this comment

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

LGTM.

@freeznet freeznet merged commit caeb054 into master Nov 2, 2022
@freeznet freeznet deleted the add_batch_source_config branch November 2, 2022 04:09
@freeznet
Copy link
Member

freeznet commented Nov 2, 2022

please remember to add the support to mesh worker service as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-added m/2022-11 type/feature Indicates new functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] function mesh is not friendly to batch source configuration.
5 participants