pulsar icon indicating copy to clipboard operation
pulsar copied to clipboard

[improve][doc] Add specified_positions_divide split algorithm in doc

Open Demogorgon314 opened this issue 2 years ago • 4 comments

Motivation

#13796 added specified_positions_divide split algorithm, but some docs are missing.

Documentation

Check the box below or label this PR directly.

Need to update docs?

  • [ ] doc-required (Your PR needs to update docs, and you will update later)

  • [ ] doc-not-needed (Please explain why)

  • [x] doc (Your PR contains doc changes)

  • [ ] doc-complete (Docs have been already added)

Demogorgon314 avatar Jun 30 '22 03:06 Demogorgon314

/pulsarbot run-failure-checks

Demogorgon314 avatar Jun 30 '22 07:06 Demogorgon314

/pulsarbot run-failure-checks

Demogorgon314 avatar Jun 30 '22 15:06 Demogorgon314

/pulsarbot run-failure-checks

dave2wave avatar Jul 08 '22 20:07 dave2wave

The pr had no activity for 30 days, mark with Stale label.

github-actions[bot] avatar Aug 08 '22 02:08 github-actions[bot]

Codecov Report

Merging #16293 (40dfa70) into master (1b5722d) will increase coverage by 1.51%. The diff coverage is 90.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #16293      +/-   ##
============================================
+ Coverage     46.34%   47.86%   +1.51%     
- Complexity    10394    10672     +278     
============================================
  Files           703      703              
  Lines         68838    68845       +7     
  Branches       7379     7382       +3     
============================================
+ Hits          31905    32953    +1048     
+ Misses        33324    32195    -1129     
- Partials       3609     3697      +88     
Flag Coverage Δ
unittests 47.86% <90.00%> (+1.51%) :arrow_up:

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

Impacted Files Coverage Δ
...ookkeeper/mledger/impl/ManagedCursorContainer.java 76.80% <85.71%> (+0.76%) :arrow_up:
...r/service/AbstractDispatcherMultipleConsumers.java 48.38% <100.00%> (+2.62%) :arrow_up:
...ava/org/apache/pulsar/broker/service/Consumer.java 71.35% <100.00%> (+2.31%) :arrow_up:
...lsar/broker/loadbalance/impl/ThresholdShedder.java 3.27% <0.00%> (-27.87%) :arrow_down:
...ker/loadbalance/impl/LeastLongTermMessageRate.java 73.33% <0.00%> (-20.00%) :arrow_down:
...rg/apache/pulsar/broker/lookup/v1/TopicLookup.java 60.00% <0.00%> (-13.34%) :arrow_down:
...org/apache/pulsar/broker/loadbalance/LoadData.java 58.33% <0.00%> (-8.34%) :arrow_down:
...roker/loadbalance/impl/ModularLoadManagerImpl.java 59.39% <0.00%> (-4.87%) :arrow_down:
...a/org/apache/pulsar/client/impl/RawReaderImpl.java 83.90% <0.00%> (-1.15%) :arrow_down:
...rg/apache/pulsar/proxy/server/ProxyConnection.java 55.21% <0.00%> (-1.13%) :arrow_down:
... and 65 more

codecov-commenter avatar Dec 10 '22 15:12 codecov-commenter

Merging...

Thanks for taking care of this!

tisonkun avatar Dec 11 '22 00:12 tisonkun