Skip to content

[SPARK-19749][SS] Name socket source with a meaningful name#17082

Closed
uncleGen wants to merge 1 commit intoapache:masterfrom
uncleGen:SPARK-19749
Closed

[SPARK-19749][SS] Name socket source with a meaningful name#17082
uncleGen wants to merge 1 commit intoapache:masterfrom
uncleGen:SPARK-19749

Conversation

@uncleGen
Copy link
Contributor

What changes were proposed in this pull request?

Name socket source with a meaningful name

How was this patch tested?

Jenkins

@srowen
Copy link
Member

srowen commented Feb 27, 2017

OK, how about other sources? it'd be nice to given them toString methods consistently along with other related classes, if you're bothering to do one.

@uncleGen
Copy link
Contributor Author

@srowen I think this is the only one souce forgotten to name.

@SparkQA
Copy link

SparkQA commented Feb 27, 2017

Test build #73499 has finished for PR 17082 at commit 68349fa.

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

@zsxwing
Copy link
Member

zsxwing commented Feb 28, 2017

Thanks! LGTM. Merging to master.

@asfgit asfgit closed this in 7353038 Feb 28, 2017
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.

4 participants