Skip to content
This repository has been archived by the owner on Jan 20, 2022. It is now read-only.

Remove redundant .unit call #525

Closed
johnynek opened this issue Jun 30, 2014 · 0 comments · Fixed by #526
Closed

Remove redundant .unit call #525

johnynek opened this issue Jun 30, 2014 · 0 comments · Fixed by #526

Comments

@johnynek
Copy link
Collaborator

https://github.com/twitter/summingbird/blob/develop/summingbird-online/src/main/scala/com/twitter/summingbird/online/executor/AsyncBase.scala#L82

That is just wasting memory.
johnynek added a commit that referenced this issue Jul 2, 2014
@johnynek
Fix #525, remove redundant Future.unit call
95ca1bf
@johnynek johnynek mentioned this issue Jul 2, 2014
Merged
snoble pushed a commit to snoble/summingbird that referenced this issue Sep 8, 2017
@johnynek
Merge pull request twitter#525 from twitter/jnievelt/524
dd80866 
Fixing BytesSpec to not reference head on an empty list
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Removes redundant Future.unit call twitter/summingbird
1 participant
@johnynek