Skip to content

Conversation

@pulltab
Copy link
Contributor

@pulltab pulltab commented Jun 12, 2018

No description provided.

Brian L. Troutwine and others added 8 commits May 22, 2018 12:28
This commit contains a small update to the Dockerfile but is
otherwise just here to bump the Cargo.toml version number.

Signed-off-by: Brian L. Troutwine <blt@postmates.com>
This commit removes the Kinesis and Firehose sinks, long since disused
by and cernan users I was able to find. The primary motivation is not
the lack of use but the difficulty of upgrading to a new version of
rusoto as we've let this sit for a while.

This commit opens the 0.9.0 series. These sinks will continue to be
supported in the 0.8 series.

Signed-off-by: Brian L. Troutwine <blt@postmates.com>
I'm hunting a leak folks are reporting in k8s-land. The leak, maybe, is
a result of us tracking file limits by lines, rather than by specific byte
totals. The configuration of the file source is now adjusted to use
'max_read_bytes' and not 'max_lines_read' as in 0.8.

Also, I have now removed the explicit dependency on hyper. We rely
entirely on reqwest in the influxdb sink.

Signed-off-by: Brian L. Troutwine <blt@postmates.com>
This commit contains suggestions from clippy, no actual changes
or leak results.

I'm being told that the file change, while helpful, is not resolving
the issue. Which, that implies some other area of unbounded growth.

Signed-off-by: Brian L. Troutwine <blt@postmates.com>
The BufferedPayload reader would allocate any size the user
requested, no matter how large. This is not great, it turns out,
when the user is sending us nonsense. The payloads are now capped
at 1Mb. This should be made configurable.

Signed-off-by: Brian L. Troutwine <blt@postmates.com>
@pulltab pulltab requested a review from blt June 12, 2018 19:05
@codecov
Copy link

codecov bot commented Jun 12, 2018

Codecov Report

Merging #445 into stable will increase coverage by 0.26%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           stable     #445      +/-   ##
==========================================
+ Coverage   91.83%   92.09%   +0.26%     
==========================================
  Files          31       29       -2     
  Lines        4162     4037     -125     
==========================================
- Hits         3822     3718     -104     
+ Misses        340      319      -21

@pulltab pulltab merged commit 49b198f into stable Jun 12, 2018
@pulltab pulltab deleted the release-0.9.0 branch June 12, 2018 20:10
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.

4 participants