[GitHub] [spark] LuciferYang commented on pull request #43193: Replace `Class.newInstance()` with `Class.getDeclaredConstructor().newInstance()`

["LuciferYang (via GitHub)" <gi...@apache.org>]

LuciferYang commented on PR #43193:
URL: https://github.com/apache/spark/pull/43193#issuecomment-1741973611

   Good point~ one approach is to add a Guava Cache to cache these no-argument Constructors to alleviate this problem, another approach is to maintain the status quo and continue to use the `.newInstance()` API for now.


-- 
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.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org