Skip to content

Conversation

@scwf
Copy link
Contributor

@scwf scwf commented Apr 27, 2015

rename DataTypeParser.apply to DataTypeParser.parse to make it more clear and readable.
/cc @rxin

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this asInstanceOf is not necessary, also delete it here

@SparkQA
Copy link

SparkQA commented Apr 27, 2015

Test build #30960 has finished for PR 5710 at commit c319977.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.
  • This patch does not change any dependencies.

@rxin
Copy link
Contributor

rxin commented Apr 27, 2015

Merging in master. Thanks!

@asfgit asfgit closed this in d188b8b Apr 27, 2015
@scwf scwf deleted the apply branch April 27, 2015 04:31
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request May 14, 2015
rename DataTypeParser.apply to DataTypeParser.parse to make it more clear and readable.
/cc rxin

Author: wangfei <wangfei1@huawei.com>

Closes apache#5710 from scwf/apply and squashes the following commits:

c319977 [wangfei] rename apply to parse
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
rename DataTypeParser.apply to DataTypeParser.parse to make it more clear and readable.
/cc rxin

Author: wangfei <wangfei1@huawei.com>

Closes apache#5710 from scwf/apply and squashes the following commits:

c319977 [wangfei] rename apply to parse
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