[CORE]codeStyle: uniform ConcurrentHashMap define in StorageLevel.scala with other places

[liyezhang556520]

No description provided.

[SparkQA]

QA tests have started for PR 2793 at commit 5884735.

• This patch merges cleanly.

[srowen]

This also seems at about the level of too trivial to bother with. It replaces a single fully qualified class name with an import, which doesn't even simplify.

[liyezhang556520]

But what about other places use ConcurrentHashMap, seems all of them are using import, should not they be unified?

[srowen]

That's not what the PR does though. How many occurrences are there to change?

[liyezhang556520]

Well, only here use a single fully qualified class name, and other places use import. like line 25 in SortShuffleManager.scala, line 602 in Utils.scala, etc.

[SparkQA]

QA tests have finished for PR 2793 at commit 5884735.

• 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/21720/
Test PASSed.

[pwendell]

meh - seems fine to me as a minor cleanup. Jenkins, test this please. If it passes we can merge it.

[SparkQA]

Test build #23602 has started for PR 2793 at commit 5884735.

• This patch merges cleanly.

[SparkQA]

Test build #23602 has finished for PR 2793 at commit 5884735.

• 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/23602/
Test PASSed.

[liyezhang556520]

@pwendell , can this be merged?

[JoshRosen]

Meh, I guess we might as well merge this if it still passes tests.

Jenkins, retest this please.

[liyezhang556520]

Jenkins, retest this please

[SparkQA]

Test build #24344 has started for PR 2793 at commit 5884735.

• This patch merges cleanly.

[SparkQA]

Test build #24344 has finished for PR 2793 at commit 5884735.

• 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/24344/
Test PASSed.

[JoshRosen]

Alright, I've merged this into master. Thanks!