Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Range OnSubscribe#841

Merged
benjchristensen merged 2 commits intoReactiveX:masterfrom
benjchristensen:operator-range
Feb 9, 2014
Merged

Range OnSubscribe#841
benjchristensen merged 2 commits intoReactiveX:masterfrom
benjchristensen:operator-range

Conversation

@benjchristensen
Copy link
Member

Reimplemented as a simple loop instead of Iterable.

Simpler code gives a performance bump:

     * -- 0.17
     * 
     * Run: 10 - 271,147,198 ops/sec
     * Run: 11 - 274,821,481 ops/sec
     * Run: 12 - 271,632,295 ops/sec
     * Run: 13 - 277,876,014 ops/sec
     * Run: 14 - 274,821,763 ops/sec
     * 
     * -- 0.16.1
     * 
     * Run: 10 - 222,104,280 ops/sec
     * Run: 11 - 224,311,761 ops/sec
     * Run: 12 - 222,999,339 ops/sec
     * Run: 13 - 222,344,174 ops/sec
     * Run: 14 - 225,247,983 ops/sec

benjchristensen added a commit that referenced this pull request Feb 9, 2014
@benjchristensen benjchristensen merged commit 185a575 into ReactiveX:master Feb 9, 2014
@benjchristensen benjchristensen deleted the operator-range branch February 9, 2014 03:25
jihoonson pushed a commit to jihoonson/RxJava that referenced this pull request Mar 6, 2020
…nException, but instead the cause should be wrapped. (ReactiveX#841)

Fixed FixedThreadPoolBulkhead: A CompletionException should not be wrapped inside of another CompletionException.
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.

1 participant