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

add API for retrieving topic list from kafka manager #137

Merged
merged 4 commits into from
Nov 24, 2015

Conversation

jisookim0513
Copy link
Contributor

No description provided.

@jisookim0513
Copy link
Contributor Author

@patelh the build fails with updating logkafka config test, but the test passes on my local machine. What could be the difference in the testing environment?

@patelh
Copy link
Collaborator

patelh commented Oct 18, 2015

Yea, non-deterministic test, so it could fail depending on timing.

-Hiral

On Tue, Oct 13, 2015 at 12:13 PM, Jisoo Kim notifications@github.com
wrote:

@patelh https://github.com/patelh the build fails with updating
logkafka config test, but the test passes on my local machine. What could
be the difference in the testing environment?


Reply to this email directly or view it on GitHub
#137 (comment).

@patelh
Copy link
Collaborator

patelh commented Nov 11, 2015

can you rebase this?

@jisookim0513
Copy link
Contributor Author

@patelh rebased it

patelh added a commit that referenced this pull request Nov 24, 2015
add API for retrieving topic list from kafka manager
@patelh patelh merged commit 1289f2a into yahoo:master Nov 24, 2015
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.

2 participants