concatArrayEagerDelayError documented but missing? 2.16

[AndreasBackx]

On 2.1.16 it is documented in the wiki and on readthedocs that concatArrayEagerDelayError is new in RxJava 2 together with:

• concatArray
• concatArrayDelayError
• concatArrayEager

Those are all there, but concatArrayEagerDelayError doesn't seem to be implemented:

Observable.concatArrayEagerDelayError()

Am I perhaps missing something?

[akarnokd]

Looks like concatArrayEagerDelayError has not been exposed so far: https://github.com/ReactiveX/RxJava/blob/2.x/src/main/java/io/reactivex/Observable.java#L1341

[AndreasBackx]

Yes, I've overlooked that, might've been because I'm on my laptop right now and it's chugging along slowly on those large files on GitHub.

Is the plan to expose this in the future?

[akarnokd]

No plans and there was no feature request for them. It would require 6~8 new methods:

{Flowable|Observable}.concatArrayEagerDelayError([tillTheEnd[, maxConcurrency [, prefetch]]], sources...)

and at least 2 diagrams per base class.

[AndreasBackx]

I'm up for providing a PR if that's welcome. Thank you in advance for providing me with the methods that would need to be implemented. I haven't dabbled into the RxJava source, but I guess I can figure it out. What would I need to create the diagrams?

Looking at the tests for concatArrayEager, and concatArrayDelayError. I assume that I only have to implement the tests like concatArrayDelay but for concatArrayEagerDelayError.

[akarnokd]

I draw them in Excel but others use OmniGraffle or Inkscape.

You can try making a PR.

[AndreasBackx]

Small update. I was on holiday and am currently studying for exams for another month so it'll be more than a month till I can get to this unfortunately.

[akarnokd]

I've done it and the necessary marbles in #6143.

[AndreasBackx]

Too bad for me, I was looking forward to it. :(

Great to have it exposed now though.