Skip to content

Conversation

@dongjinleekr
Copy link
Contributor

What changes were proposed in this pull request?

Fix unused imports & outdated comments on kafka-0-10-sql module. (Found while I was working on SPARK-23539)

How was this patch tested?

Existing unit tests.

@dongjinleekr dongjinleekr changed the title Feature/fix kafka sql trivials [MINOR][SS] Fix kafka-0-10-sql trivials Sep 5, 2018
@SparkQA

This comment has been minimized.

@dongjinleekr dongjinleekr force-pushed the feature/fix-kafka-sql-trivials branch from f116d9e to ee4e97a Compare September 5, 2018 13:30
@SparkQA
Copy link

SparkQA commented Sep 5, 2018

Test build #95719 has finished for PR 22342 at commit ee4e97a.

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

@srowen
Copy link
Member

srowen commented Sep 7, 2018

It's fine. We usually don't do little minor cleanup PRs but it's reasonable for a new contributor to get warmed up like this.

@srowen
Copy link
Member

srowen commented Sep 7, 2018

Merged to master/2.4

asfgit pushed a commit that referenced this pull request Sep 7, 2018
## What changes were proposed in this pull request?

Fix unused imports & outdated comments on `kafka-0-10-sql` module. (Found while I was working on [SPARK-23539](#22282))

## How was this patch tested?

Existing unit tests.

Closes #22342 from dongjinleekr/feature/fix-kafka-sql-trivials.

Authored-by: Lee Dongjin <[email protected]>
Signed-off-by: Sean Owen <[email protected]>
(cherry picked from commit 458f501)
Signed-off-by: Sean Owen <[email protected]>
@asfgit asfgit closed this in 458f501 Sep 7, 2018
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