[SPARK-5914] to run spark-submit requiring only user perm on windows

[judynash]

Because windows on-default does not grant read permission to jars except to admin, spark-submit would fail with "ClassNotFound" exception if user runs slave service with only user permission.
This fix is to add read permission to owner of the jar (which would be the slave service account in windows )

[AmplabJenkins]

Can one of the admins verify this patch?

[pwendell]

mind adding a space after if?

[pwendell]

Jenkins, test this please.

[SparkQA]

Test build #27888 has started for PR 4742 at commit 1de3c0e.

• This patch merges cleanly.

[srowen]

This doesn't need to be in both branches of the if right -- this boils down to just adding u+r if isWindows and not affecting the current line.

[SparkQA]

Test build #27888 has finished for PR 4742 at commit 1de3c0e.

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

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27888/
Test PASSed.

[judynash]

Looks like the test didn't rerun after submitting updated code?

[srowen]

Jenkins, retest this please

[SparkQA]

Test build #27968 has started for PR 4742 at commit e288e56.

• This patch merges cleanly.

[SparkQA]

Test build #27968 has finished for PR 4742 at commit e288e56.

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

[AmplabJenkins]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27968/
Test PASSed.

[srowen]

LGTM. I'll fix the space before the brace on merge. This seems like a clean, simple fix.