Fixes SPARK-8730 - Deser objects containing a primitive class attribute

[EugenCepoi]

No description provided.

[SparkQA]

Test build #36127 has finished for PR 7122 at commit 66dbc80.

• This patch fails Scala style tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #36128 has finished for PR 7122 at commit 42244d6.

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

[squito]

wrap the try block w/ braces eg

try {
   Class.forName(...)
} catch {
 ...

[SparkQA]

Test build #38351 has finished for PR 7122 at commit 9cfc942.

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

[squito]

can you rename this to match the class, JavaDeserializationStream? Alternatively we could give it some name which is more specific, eg. JavaSerializationPrimitiveClassHelper or something, but I think just making it a companion object is good enough.

[EugenCepoi]

good point, I hesitated to do it at first but don't remember why I decided to not put it in a companion object. Will do

[SparkQA]

Test build #38394 has finished for PR 7122 at commit 24e3d99.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[EugenCepoi]

@squito I don't get what fails. Is it some random issue or did I really break something? If I didn't should we just re run the tests?

[SparkQA]

Test build #41828 has finished for PR 7122 at commit 5e50c62.

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

[squito]

@EugenCepoi yikes, sorry I forgot about this for so long! thanks for brining it up to date, merging to master

[EugenCepoi]

Np, thanks for the review :)