Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@luis3m
Copy link
Contributor

@luis3m luis3m commented May 17, 2020

Resolves #3602

@luis3m luis3m requested a review from iravid as a code owner May 17, 2020 17:05
@iravid iravid added enhancement New feature or request stream ZIO Stream labels May 18, 2020
@reibitto
Copy link
Contributor

I think that fixes that particular issue. 👍

However, I think there's something else:

ZStream("a", "b", "c").debounce(1.second).runCollect

This returns all the elements when I'd expect it to only return c. I think if you receive a chunk with more than 1 element you only want to return the last element from it, right?

@luis3m
Copy link
Contributor Author

luis3m commented May 19, 2020

@reibitto yes, I mentioned this particular difference in the previous PR I think. I can change this.

@iravid
Copy link
Member

iravid commented May 21, 2020

@luis3m Do you want to address @reibitto's comment here or in a follow-up?

@luis3m
Copy link
Contributor Author

luis3m commented May 21, 2020

@iravid preferably a follow-up

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request stream ZIO Stream

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ZStream.debounce emit at end of stream

3 participants