Skip to content

Implemented Sample Operation#248

Merged
benjchristensen merged 9 commits intoReactiveX:masterfrom
jmhofer:sample
May 1, 2013
Merged

Implemented Sample Operation#248
benjchristensen merged 9 commits intoReactiveX:masterfrom
jmhofer:sample

Conversation

@jmhofer
Copy link
Contributor

@jmhofer jmhofer commented Apr 27, 2013

Now that OperationInterval is in, OperationSample (see #74) was kind of straightforward. I also added a unit test against sampling, and allowed for specifying a scheduler used by the sample operation.

@cloudbees-pull-request-builder

RxJava-pull-requests #108 SUCCESS
This pull request looks good

@benjchristensen
Copy link
Member

We want methods on Observable don't we?

@jmhofer
Copy link
Contributor Author

jmhofer commented May 1, 2013

Sure, I just completely forgot about that. I'll add them.

@cloudbees-pull-request-builder

RxJava-pull-requests #113 SUCCESS
This pull request looks good

benjchristensen added a commit that referenced this pull request May 1, 2013
Implemented Sample Operation
@benjchristensen benjchristensen merged commit 1aa722d into ReactiveX:master May 1, 2013
@jmhofer jmhofer deleted the sample branch May 1, 2013 21:48
rickbw pushed a commit to rickbw/RxJava that referenced this pull request Jan 9, 2014
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.

3 participants