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 #5939

Merged
merged 3 commits into from
Oct 25, 2023
Merged

Topic properties support for update and remove #5939

merged 3 commits into from
Oct 25, 2023

Conversation

liangyepianzhou
Copy link

@liangyepianzhou liangyepianzhou commented Oct 25, 2023 •
edited
Loading

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
Topic properties support for update and remove (apache#5911)
bc7ee1a 
(cherry picked from commit 6d7dfcf)
(cherry picked from commit 6564e6e)
@liangyepianzhou
[fix][broker] managedLedger.getConfig().getProperties().putAll(proper…
8765134 
…ties) NPE (apache#20361) (apache#5932)

(cherry picked from commit aa7decc)

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

CI test:liangyepianzhou#4

@liangyepianzhou
Revert gpg and delete useless file
73b7c06 
(cherry picked from commit 2739f2c)
@liangyepianzhou liangyepianzhou merged commit b67024d into streamnative:branch-2.10.5.6-5ce22c Oct 25, 2023
1 check passed
@liangyepianzhou liangyepianzhou deleted the properties branch October 25, 2023 01:56
@liangyepianzhou liangyepianzhou restored the properties branch October 25, 2023 07:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@liangyepianzhou