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.

Sign up for GitHub

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

Jump to bottom

Meassure report sizes #1458

Merged
merged 1 commit into from
May 10, 2016
Merged

Meassure report sizes #1458

merged 1 commit into from
May 10, 2016

Conversation

2opremio
Copy link
Contributor

Helps debugging issues like #1201 and #985

@2opremio 2opremio force-pushed the meassure-report-sizes branch 5 times, most recently from bde8d7a to 33321e3 Compare May 10, 2016 07:46
@tomwilkie
Copy link
Contributor

Wouldn't it be easier to read the Content-Length header?

@2opremio
Copy link
Contributor Author

Wouldn't it be easier to read the Content-Length header?

For the compressed size it would, but we can't for the uncompressed size and I like the uniformity.

@2opremio 2opremio force-pushed the meassure-report-sizes branch from 33321e3 to 64bf6d3 Compare May 10, 2016 09:32
@2opremio 2opremio force-pushed the meassure-report-sizes branch from 64bf6d3 to 3cf3c71 Compare May 10, 2016 09:45
@paulbellamy paulbellamy merged commit 7a58e11 into master May 10, 2016
@paulbellamy paulbellamy deleted the meassure-report-sizes branch May 10, 2016 10:27
@2opremio 2opremio mentioned this pull request Jul 26, 2016
Closed
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@2opremio @tomwilkie @paulbellamy