Skip to content
This repository has been archived by the owner on Jul 31, 2023. It is now read-only.

Add Flush to force aggregate in-memory buffer #923

Closed
wants to merge 1 commit into from

Conversation

rakyll
Copy link
Contributor

@rakyll rakyll commented Sep 21, 2018

Fixes #862.

@odeke-em
Copy link
Member

@rakyll thanks for also working on this! I had mailed #922 sometime back. However, be ware that calling Flush after any future stops shouldn't block :)

@bogdandrutu
Copy link
Contributor

@rakyll I think (RetrieveData)[https://github.com/census-instrumentation/opencensus-go/blob/master/stats/view/worker.go#L94] is your answer for this. See how I used it https://github.com/census-instrumentation/opencensus-service/blob/master/observability/observabilitytest/observabilitytest.go#L105

Do you think that is enough?

@wdhnl
Copy link

wdhnl commented May 3, 2019

Can this be looked at again?

@rakyll
Copy link
Contributor Author

rakyll commented Mar 23, 2021

Closing this due to inactivity, please open another PR if it needs to be addressed.

@rakyll rakyll closed this Mar 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

stats/view: add Flush to immediately report all collected data points regardless of buffering
4 participants