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

[INLONG-11544][Manager] Optimize the configuration of the Manager schedule module #11545

Merged
merged 1 commit into from
Nov 26, 2024

Conversation

Zkplo
Copy link
Contributor

@Zkplo Zkplo commented Nov 25, 2024

Fixes #11544

Motivation

Modifications

  • Simplify Annotations.
  • Add default values.

Verifying this change

(Please pick either of the following options)

  • This change is a trivial rework/code cleanup without any test coverage.

  • This change is already covered by existing tests, such as:
    (please describe tests)

  • This change added tests and can be verified as follows:

    (example:)

    • Added integration tests for end-to-end deployment with large payloads (10MB)
    • Extended integration test for recovery after broker failure

@Zkplo Zkplo force-pushed the INLONG-11544 branch 2 times, most recently from 53d1c0d to 98629f6 Compare November 25, 2024 15:11
@Zkplo Zkplo force-pushed the INLONG-11544 branch 3 times, most recently from 1de91d8 to d8bffde Compare November 25, 2024 15:58
dockerzhang
dockerzhang previously approved these changes Nov 26, 2024
@dockerzhang dockerzhang merged commit 4c475e9 into apache:master Nov 26, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature][manager] Optimize the configuration of the Manager schedule module
4 participants