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

Integration test and documentation for Proxy w/o ZK #3352

Merged
merged 1 commit into from
Jan 11, 2019
Merged

Integration test and documentation for Proxy w/o ZK #3352

merged 1 commit into from
Jan 11, 2019

Conversation

ivankelly
Copy link
Contributor

Adds an integration test which creates a proxy which connects to the
brokers using broker URLs rather than service discovery.

Adds documentation for configuring the proxy in this way.

Issue: #2405

@ivankelly
Integration test and documentation for Proxy w/o ZK
019b8fb 
Adds an integration test which creates a proxy which connects to the
brokers using broker URLs rather than service discovery.

Adds documentation for configuring the proxy in this way.

Issue: apache#2405
@ivankelly ivankelly added doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. area/test type/cleanup Code or doc cleanups e.g. remove the outdated documentation or remove the code no longer in use area/proxy labels Jan 10, 2019
@ivankelly ivankelly self-assigned this Jan 10, 2019
merlimat
merlimat approved these changes Jan 10, 2019
View reviewed changes
Copy link
Contributor

@merlimat merlimat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@merlimat merlimat added this to the 2.3.0 milestone Jan 11, 2019
@merlimat merlimat merged commit aa3fd10 into apache:master Jan 11, 2019
@sijie sijie mentioned this pull request May 19, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@merlimat merlimat merlimat approved these changes

@sijie sijie Awaiting requested review from sijie

@jai1 jai1 Awaiting requested review from jai1

Assignees

@ivankelly ivankelly

Labels
area/proxy area/test doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. type/cleanup Code or doc cleanups e.g. remove the outdated documentation or remove the code no longer in use
Projects
None yet
Milestone
2.3.0
Development

Successfully merging this pull request may close these issues.
2 participants
@ivankelly @merlimat