Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

3.x: Flowable scan/scanWith backpressure documentation update #7110

Merged
merged 2 commits into from
Nov 12, 2020
Merged

3.x: Flowable scan/scanWith backpressure documentation update #7110

merged 2 commits into from
Nov 12, 2020

Conversation

fsbarata
Copy link
Contributor

@fsbarata fsbarata commented Nov 12, 2020

Documented Flowable initial value scan variants upstream consumption pattern as it is different from the variant with no initial value.

Resolves #7109

Documented Flowable initial value scan variants upstream consumption pattern as it is different from the variant with no initial value.
@@ -14736,6 +14736,8 @@ public final void safeSubscribe(@NonNull Subscriber<@NonNull ? super T> subscrib
* <dt><b>Backpressure:</b></dt>
* <dd>The operator honors downstream backpressure and expects the current {@code Flowable} to honor backpressure as well.
* Violating this expectation, a {@link MissingBackpressureException} <em>may</em> get signaled somewhere downstream.</dd>
* <dd>The downstream request pattern is not preserved across this operator.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You don't have to start a new <dd> here, just add the text on a new line inside the previous <dd>.

Copy link
Contributor Author

@fsbarata fsbarata Nov 12, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • merge <dd> sections

@codecov
Copy link

codecov bot commented Nov 12, 2020

Codecov Report

Merging #7110 (e416b6f) into 3.x (57fd72e) will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##                3.x    #7110      +/-   ##
============================================
+ Coverage     99.51%   99.53%   +0.01%     
  Complexity     6670     6670              
============================================
  Files           742      742              
  Lines         47275    47275              
  Branches       6374     6374              
============================================
+ Hits          47048    47054       +6     
  Misses          104      104              
+ Partials        123      117       -6     
Impacted Files Coverage Δ Complexity Δ
.../main/java/io/reactivex/rxjava3/core/Flowable.java 100.00% <ø> (ø) 595.00 <0.00> (ø)
.../operators/flowable/FlowableBlockingSubscribe.java 93.02% <0.00%> (-4.66%) 10.00% <0.00%> (-1.00%)
...eactivex/rxjava3/processors/BehaviorProcessor.java 97.48% <0.00%> (-2.52%) 50.00% <0.00%> (ø%)
...tivex/rxjava3/internal/jdk8/ParallelCollector.java 91.74% <0.00%> (-1.84%) 2.00% <0.00%> (ø%)
...rnal/operators/flowable/FlowableFlatMapSingle.java 94.18% <0.00%> (-1.17%) 2.00% <0.00%> (ø%)
...ava3/internal/operators/maybe/MaybeMergeArray.java 98.87% <0.00%> (-1.13%) 6.00% <0.00%> (ø%)
...ternal/operators/observable/ObservableGroupBy.java 98.90% <0.00%> (-1.10%) 2.00% <0.00%> (ø%)
...perators/observable/ObservableMergeWithSingle.java 99.05% <0.00%> (-0.95%) 2.00% <0.00%> (ø%)
.../operators/observable/ObservableCombineLatest.java 99.38% <0.00%> (-0.62%) 6.00% <0.00%> (ø%)
...3/internal/operators/flowable/FlowableGroupBy.java 85.02% <0.00%> (-0.30%) 3.00% <0.00%> (ø%)
... and 11 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 57fd72e...e416b6f. Read the comment docs.

@akarnokd akarnokd merged commit 78b29f7 into ReactiveX:3.x Nov 12, 2020
@akarnokd
Copy link
Member

Thanks!

@fsbarata
Copy link
Contributor Author

My pleasure!

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

Successfully merging this pull request may close these issues.

Flowable scan operators backpressure inconsistency
2 participants