Skip to content
This repository has been archived by the owner on Apr 1, 2024. It is now read-only.

Topic properties support for update and remove #5940

Merged

Conversation

liangyepianzhou
Copy link

Motivation

Topic properties support for update and remove

Verifying this change

  • Make sure that the change passes the CI checks.

(Please pick either of the following options)

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

(or)

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

(or)

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

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

  • Dependencies (add or upgrade a dependency)
  • The public API
  • The schema
  • The default values of configurations
  • The threading model
  • The binary protocol
  • The REST endpoints
  • The admin CLI options
  • The metrics
  • Anything that affects deployment

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository:

liangyepianzhou and others added 3 commits October 24, 2023 17:27
(cherry picked from commit 6d7dfcf)
(cherry picked from commit 6564e6e)
…ties) NPE (apache#20361) (apache#5932)

(cherry picked from commit aa7decc)

Co-authored-by: Heesung Sohn <[email protected]>
(cherry picked from commit 5f6ed1b)
(cherry picked from commit 2739f2c)
@liangyepianzhou
Copy link
Author

CI test: liangyepianzhou#4

@codelipenghui codelipenghui merged commit cc221ee into streamnative:branch-2.10.5.6-5ce22c Oct 25, 2023
1 of 3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants