Remove unicode operator from RDD.scala

[dorx]

Some IDEs don’t support unicode characters in source code. Check if this breaks binary compatibility.

[dorx]

@ash211 @rxin @mengxr

[AmplabJenkins]

Merged build triggered.

[AmplabJenkins]

Merged build started.

[rxin]

@ash211 ...

[AmplabJenkins]

Merged build finished. All automated tests passed.

[AmplabJenkins]

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

[ash211]

That even works in Scala? I definitely didn't intend to commit that as a single Unicode character as opposed to two characters -- an '=' and a '>' and code in Vim which I wouldn't expect to do character replacement, but who knows.

Definitely replace it with =>

+1 LGTM

[rxin]

Thanks. I've merged this.

[dorx]

@ash211 FWIW they have other unicode operators supported in addition to ⇒: http://www.scala-lang.org/old/node/4723
Might want to look into adding a no-unicode-char rule in scalastyle.

[ash211]

Agreed, I'll file a ticket about it

On Wed, Jun 18, 2014 at 6:07 PM, dorx [email protected] wrote:

@ash211 https://github.com/ash211 FWIW they have other unicode
operators supported in addition to ⇒:
http://www.scala-lang.org/old/node/4723
Might want to look into adding a no-unicode-char rule in scalastyle.

—
Reply to this email directly or view it on GitHub
#1119 (comment).

[ash211]

Dorx, can you add what IDE that was to the below ticket? Thanks!

https://issues.apache.org/jira/browse/SPARK-2182

On Wed, Jun 18, 2014 at 6:18 PM, Andrew Ash [email protected] wrote:

Agreed, I'll file a ticket about it

On Wed, Jun 18, 2014 at 6:07 PM, dorx [email protected] wrote:

@ash211 https://github.com/ash211 FWIW they have other unicode
operators supported in addition to ⇒:
http://www.scala-lang.org/old/node/4723
Might want to look into adding a no-unicode-char rule in scalastyle.

—
Reply to this email directly or view it on GitHub
#1119 (comment).