Skip to content

SPARK-1523: improve the readability of code in AkkaUtil - #434

Closed
CodingCat wants to merge 1 commit into
apache:masterfrom
CodingCat:SPARK-1523
Closed

SPARK-1523: improve the readability of code in AkkaUtil#434
CodingCat wants to merge 1 commit into
apache:masterfrom
CodingCat:SPARK-1523

Conversation

@CodingCat

Copy link
Copy Markdown
Contributor

Actually it is separated from #85 as suggested by @rxin

compare

https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/util/AkkaUtils.scala#L122

and

https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/util/AkkaUtils.scala#L117

the first one use get and then toLong, the second one getLong....better to make them consistent

very very small fix........

@AmplabJenkins

Copy link
Copy Markdown

Merged build triggered.

@AmplabJenkins

Copy link
Copy Markdown

Merged build started.

@AmplabJenkins

Copy link
Copy Markdown

Merged build finished. All automated tests passed.

@AmplabJenkins

Copy link
Copy Markdown

All automated tests passed.
Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/14208/

@rxin

rxin commented Apr 18, 2014

Copy link
Copy Markdown
Contributor

I merged this. Thanks.

asfgit pushed a commit that referenced this pull request Apr 18, 2014
Actually it is separated from #85 as suggested by @rxin
compare
https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/util/AkkaUtils.scala#L122
and
https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/util/AkkaUtils.scala#L117
the first one use get and then toLong, the second one getLong....better to make them consistent
very very small fix........
Author: CodingCat <zhunansjtu@gmail.com>
Closes#434 from CodingCat/SPARK-1523 and squashes the following commits:
0e86f3f [CodingCat] improve the readability of code in AkkaUtil
(cherry picked from commit 3c7a9ba)
Signed-off-by: Reynold Xin <rxin@apache.org>
pwendell added a commit to pwendell/spark that referenced this pull request May 12, 2014
Fixed SVDPlusPlusSuite in Maven build.
This should go into 0.9.0 also.
pdeyhim pushed a commit to pdeyhim/spark-1 that referenced this pull request Jun 25, 2014
Actually it is separated from apache#85 as suggested by @rxin
compare
https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/util/AkkaUtils.scala#L122
and
https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/util/AkkaUtils.scala#L117
the first one use get and then toLong, the second one getLong....better to make them consistent
very very small fix........
Author: CodingCat <zhunansjtu@gmail.com>
Closesapache#434 from CodingCat/SPARK-1523 and squashes the following commits:
0e86f3f [CodingCat] improve the readability of code in AkkaUtil
andrewor14 pushed a commit to andrewor14/spark that referenced this pull request Jan 8, 2015
Fixed SVDPlusPlusSuite in Maven build.
This should go into 0.9.0 also.
(cherry picked from commit 087487e)
Signed-off-by: Patrick Wendell <pwendell@gmail.com>
mnazbro pushed a commit to mnazbro/spark that referenced this pull request Nov 14, 2018
Fix gradle publishing to correct bintray bucket
bzhaoopenstack pushed a commit to bzhaoopenstack/spark that referenced this pull request Sep 11, 2019
arjunshroff pushed a commit to arjunshroff/spark that referenced this pull request Nov 24, 2020
Sign up for freeto 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

@CodingCat@AmplabJenkins@rxin