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

Update README for migration to travis-ci.com, and copyright #442

Merged
merged 1 commit into from Jun 8, 2018

Conversation

dparton
Copy link
Contributor

@dparton dparton commented Jun 8, 2018

No description provided.

pulltab
pulltab previously approved these changes Jun 8, 2018
@dparton dparton merged commit 4be0639 into master Jun 8, 2018
@dparton dparton deleted the update-readme branch June 8, 2018 17:20
@codecov
Copy link

codecov bot commented Jun 8, 2018

Codecov Report

Merging #442 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #442      +/-   ##
==========================================
+ Coverage   92.07%   92.09%   +0.02%     
==========================================
  Files          29       29              
  Lines        4037     4037              
==========================================
+ Hits         3717     3718       +1     
+ Misses        320      319       -1

pulltab added a commit that referenced this pull request Jun 12, 2018
* Open up 0.8.18-pre development

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>

* Remove the kinesis and firehose sink (#437)

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>

* Adjust hyper -> reqwest, introduce file source byte limits

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>

* Clippy fiddling

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>

* Cap nonblocking::BufferedPayload allocations

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>

* Explicitly set timestamps for kafka record producer (#441)

* Update README for migration to travis-ci.com, and copyright (#442)

* Bump to 0.9.0. (#443)
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.

None yet

2 participants