Skip to content

[INLONG-10017][Agent] Fix the module update configuration bug#10108

Merged
justinwwhuang merged 1 commit intoapache:masterfrom
justinwwhuang:b10017
Apr 29, 2024
Merged

[INLONG-10017][Agent] Fix the module update configuration bug#10108
justinwwhuang merged 1 commit intoapache:masterfrom
justinwwhuang:b10017

Conversation

@justinwwhuang
Copy link
Contributor

@justinwwhuang justinwwhuang commented Apr 29, 2024

Fixes #10017

Motivation

Fix the module update configuration bug

Modifications

Set the old module status to the new module

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

Documentation

No doc needed

@justinwwhuang justinwwhuang added this to the 1.13.0 milestone Apr 29, 2024
@justinwwhuang justinwwhuang self-assigned this Apr 29, 2024
Copy link
Contributor

@doleyzi doleyzi left a comment

Choose a reason for hiding this comment

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

LGTM

@justinwwhuang justinwwhuang merged commit 28bfa89 into apache:master Apr 29, 2024
@justinwwhuang justinwwhuang deleted the b10017 branch April 29, 2024 10:35
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][Sort] Definition of Sort Config

3 participants