Skip to content

Fix classifySessionsOf#542

Closed
harendra-kumar wants to merge 3 commits intotimesfrom
classifySessions
Closed

Fix classifySessionsOf#542
harendra-kumar wants to merge 3 commits intotimesfrom
classifySessions

Conversation

@harendra-kumar
Copy link
Copy Markdown
Member

No description provided.

It was running forever as the timer stream never terminated.
@harendra-kumar harendra-kumar requested a review from adithyaov May 29, 2020 06:50
Earlier the sessions that were incomplete and in the hash were dropped when the
stream ended.
-- , benchIOSink value "toPure" toPure
-- , benchIOSink value "toPureRev" toPureRev
]
, benchIOSink (value `div` 10) "classifySessionsOf"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change classifySessionsOf to classifySessionsOf/10 maybe to indicate that you're using something other than the default value. It's not a big deal though.

Copy link
Copy Markdown
Member

@adithyaov adithyaov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@harendra-kumar
Copy link
Copy Markdown
Member Author

This got merged via CLI, but github does not let us change the merge target to master. Closing.

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