Skip to content

[improve][client] Disable polling pattern topics when TopicListWatcher is enabled. - #20779

Merged
Technoboy- merged 2 commits into
apache:masterfrom
Technoboy-:disable-polling-topic
Jul 17, 2023
Merged

[improve][client] Disable polling pattern topics when TopicListWatcher is enabled.#20779
Technoboy- merged 2 commits into
apache:masterfrom
Technoboy-:disable-polling-topic

Conversation

@Technoboy-

@Technoboy- Technoboy- commented Jul 11, 2023

Copy link
Copy Markdown
Contributor

Motivation

Disable polling pattern topics task when TopicListWatcher is enabled.

The current tests have covered the cases.

Documentation

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

@Technoboy- Technoboy- self-assigned this Jul 11, 2023
@Technoboy- Technoboy- added this to the 3.1.0 milestone Jul 11, 2023
@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Jul 11, 2023
@Technoboy- Technoboy- added release/3.0.2 and removed doc-not-needed Your PR changes do not impact docs labels Jul 11, 2023
@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Jul 11, 2023
@Technoboy-
Technoboy- force-pushed the disable-polling-topic branch from 39021b5 to b7b350d Compare July 14, 2023 06:59
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #20779 (b7b350d) into master (e96b339) will increase coverage by 39.65%.
The diff coverage is 100.00%.

Impacted file tree graph

@@              Coverage Diff              @@
##             master   #20779       +/-   ##
=============================================
+ Coverage     33.44%   73.09%   +39.65%     
- Complexity    12097    32144    +20047     
=============================================
  Files          1612     1867      +255     
  Lines        126568   139056    +12488     
  Branches      13824    15300     +1476     
=============================================
+ Hits          42328   101649    +59321     
+ Misses        78686    29352    -49334     
- Partials       5554     8055     +2501     
Flag Coverage Δ
inttests 24.17% <0.00%> (+0.08%) ⬆️
systests 24.93% <100.00%> (?)
unittests 72.40% <100.00%> (+40.45%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...rg/apache/pulsar/broker/service/BrokerService.java 82.23% <ø> (+26.43%) ⬆️
...ar/client/impl/PatternMultiTopicsConsumerImpl.java 87.96% <100.00%> (+62.49%) ⬆️
...rg/apache/pulsar/client/impl/TopicListWatcher.java 67.39% <100.00%> (+44.20%) ⬆️

... and 1519 files with indirect coverage changes

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.

4 participants