forked from linkedin/brooklin
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Pull latest #1
Merged
Merged
Pull latest #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ir and reduce Histogram time (#655)
… level custom checkpoint (#653) * Add custom checkpoint datastream metadata field to override connector level custom checkpoint Add a custom checkpoint datastream metadata field which can be used to override the connector level custom checkpoint flag. * Move getCustomCheckpointing() after initializeTask()
Invoke the topic manger when the partitions are assigned
…cs prior to v4.1.0-rc2 (#658) This change restores the JMX metrics name generation logic used in io.dropwizard.metrics prior to v.4.1.0-rc2 in order to avoid breaking our existing metrics graphs and dashboards.
Fix a race condition in retrying the partition assignment
Relax Kafka broker hostname validation checks that required them to be either IPv4, IPv6, or RFC 1123-compliant. Before it got removed, this check was only performed for brokers Brooklin consumed from but not those it produced to. This check is not necessary since Kafka clients verify broker addresses through DNS resolution.
…igurable threshold (#661)
…ncrease default threshold (#664) * Add config flag to disable latency warning log in EventProducer and increase default threshold
…ctors (#668) * Add exception to warning log message on send failures for Kafka connectors * Address review comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Important: DO NOT REPORT SECURITY ISSUES DIRECTLY ON GITHUB.
For reporting security issues and contributing security fixes,
please, email security@linkedin.com instead, as described in
the contribution guidelines.
Please, take a minute to review the contribution guidelines at:
https://github.com/linkedin/Brooklin/blob/master/CONTRIBUTING.md