rednaxelafx opened a new pull request #29875:
URL: https://github.com/apache/spark/pull/29875


   ### What changes were proposed in this pull request?
   
   Use `Utils.getSimpleName` to avoid hitting `Malformed class name` error in 
`TreeNode`.
   
   ### Why are the changes needed?
   
   On older JDK versions (e.g. JDK8u), nested Scala classes may trigger 
`java.lang.Class.getSimpleName` to throw an `java.lang.InternalError: Malformed 
class name` error.
   
   Similar to https://github.com/apache/spark/pull/29050, we should use  
Spark's `Utils.getSimpleName` utility function in place of 
`Class.getSimpleName` to avoid hitting the issue.
   
   ### Does this PR introduce _any_ user-facing change?
   
   Fixes a bug that throws an error when invoking `TreeNode.nodeName`, 
otherwise no changes.
   
   ### How was this patch tested?
   
   Added new unit test case in `TreeNodeSuite`. Note that the test case assumes 
the test code can trigger the expected error, otherwise it'll skip the test 
safely, for compatibility with newer JDKs.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to