Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Log output streaming #1209

Closed
wants to merge 3 commits into from
Closed

Log output streaming #1209

wants to merge 3 commits into from

Conversation

ddollar
Copy link
Contributor

@ddollar ddollar commented Sep 9, 2016

Get logs out as fast as possible without trying to ensure that they are in the right order. We could potentially add client-side optional sorting or something like that.

Currently only apps with lots of log streams (frequently crashing processes or frequent cron) will have potentially weirdly out-of-order logs.

@codecov-io
Copy link

Current coverage is 23.51% (diff: 45.45%)

Merging #1209 into master will decrease coverage by 0.02%

@@             master      #1209   diff @@
==========================================
  Files           128        128          
  Lines         11412      11412          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits           2686       2683     -3   
- Misses         8544       8546     +2   
- Partials        182        183     +1   

Powered by Codecov. Last update f800997...b437deb

@ddollar
Copy link
Contributor Author

ddollar commented Sep 9, 2016

log streaming

@ddollar ddollar added the merge label Sep 10, 2016
@ddollar ddollar modified the milestone: 20160909 Sep 10, 2016
ddollar added a commit that referenced this pull request Sep 10, 2016
@ddollar ddollar mentioned this pull request Sep 10, 2016
18 tasks
ddollar added a commit that referenced this pull request Sep 10, 2016
@ddollar ddollar deleted the log-buffering branch September 15, 2016 20:01
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.

2 participants