Skip to content

Conversation

@koeninger
Copy link
Contributor

What changes were proposed in this pull request?

code cleanup in kafka-0-8 to match suggested changes for kafka-0-10 branch

How was this patch tested?

unit tests

@SparkQA
Copy link

SparkQA commented Jun 26, 2016

Test build #61248 has finished for PR 13908 at commit 576a0e4.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jun 26, 2016

Test build #61249 has finished for PR 13908 at commit 647c2af.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jun 26, 2016

Test build #61251 has finished for PR 13908 at commit d8a1ba0.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jun 29, 2016

Test build #61508 has finished for PR 13908 at commit ea35641.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@tdas
Copy link
Contributor

tdas commented Jun 30, 2016

LGTM. Unless you plan to make more changes.

@koeninger
Copy link
Contributor Author

I don't plan to make any more changes... I kind of doubt feedback on your #13996 is going to result in any changes that would also apply to 0.8

@tdas
Copy link
Contributor

tdas commented Jun 30, 2016

Cool. Merging this to master

@asfgit asfgit closed this in c622633 Jun 30, 2016
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.

3 participants