[SPARK-8549][SparkR] Fix the line length of SparkR

[yu-iskw]

[SPARK-8549] Fix the line length of SparkR - ASF JIRA

[SparkQA]

Test build #36462 has finished for PR 7204 at commit 1737598.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):
  • class Word2VecModel(JavaVectorTransformer, JavaSaveable, JavaLoader):

[yu-iskw]

@shivaram Could you review this PR when you have time? Thanks!

[shivaram]

can we call this storageLevelClass ?

[yu-iskw]

No. It is a just string variable.

[shivaram]

Yeah but the variable name spelling is strange right now as strage -- could you make the variable name storage instead ?

[yu-iskw]

Sorry about that. I got it.

[SparkQA]

Test build #36471 has finished for PR 7204 at commit 6fb131a.

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

[shivaram]

Thanks @yu-iskw -- Just for my information can you paste a link to output of dev/lint-r after this change ?

[yu-iskw]

@shivaram , Sure. That is the result of ./dev/lint-r at the revision d9838196ff48faeac19756852a7f695129c08047. Could you check it?
https://gist.github.com/yu-iskw/d3f414f2c18b2abc5766

[shivaram]

Thanks @yu-iskw. BTW We should investigate how to get rid of the false warnings no visible global function definition as those functions are package private.

But this change LGTM. Merging this

[yu-iskw]

@shivaram Thank you for merging it!
I agree with that. I will check the lintr code and ask Jim it.

[rxin]

@yu-iskw you might want to update your github email to include the email address you used for your commits. Otherwise it doesn't show up in the github metadata.

[yu-iskw]

@rxin Thank you for letting me know. I have updated my github email in my account profile.