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

[transaction] Use the config's metadata tenant when disable multi-tenant metadata #1980

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Demogorgon314
Copy link
Member

Motivation

When users disable the KafkaEnableMultiTenantMetadata, we should use the config's metadata tenant as the PulsarTopicProducerStateManagerSnapshotBuffer topic tenant.

Modifications

Use the config's metadata tenant when disable multi-tenant metadata

Documentation

Check the box below.

Need to update docs?

  • doc-required

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

  • no-need-doc

    (Please explain why)

  • doc

    (If this PR contains doc changes)

@codecov
Copy link

codecov bot commented Jul 27, 2023

Codecov Report

Merging #1980 (218a787) into master (e931b6d) will increase coverage by 0.04%.
Report is 1 commits behind head on master.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1980      +/-   ##
============================================
+ Coverage     17.72%   17.76%   +0.04%     
- Complexity      751      752       +1     
============================================
  Files           195      195              
  Lines         14156    14140      -16     
  Branches       1322     1321       -1     
============================================
+ Hits           2509     2512       +3     
+ Misses        11464    11446      -18     
+ Partials        183      182       -1     
Files Changed Coverage Δ
...tive/pulsar/handlers/kop/KafkaProtocolHandler.java 0.00% <0.00%> (ø)
...ative/pulsar/handlers/kop/KafkaRequestHandler.java 1.06% <ø> (+<0.01%) ⬆️
...sar/handlers/kop/storage/AppendRecordsContext.java 0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants