[GitHub] [spark] beliefer edited a comment on issue #24647: [SPARK-27776][SQL]Avoid duplicate Java reflection in DataSource.

[GitBox <gi...@apache.org>]

beliefer edited a comment on issue #24647: [SPARK-27776][SQL]Avoid duplicate Java reflection in DataSource.
URL: https://github.com/apache/spark/pull/24647#issuecomment-494214438
 
 
   > I'm not sure about the contract here, whether providers are required to be stateless. If they're not then this would be a problem for another instance that has state, or if these acquire state at some point.
   
   First of all, I am glad to see your reply.
   I have the same question as you.I investigate all the providers and found every implementation is stateless indeed. I even investigated the providers provided by third parties, such as `com.mongodb.spark.sql.DefaultSource`,`org.elasticsearch.spark.sql.DefaultSource`.
   
   Another reason I created this PR is `providingClass.getConstructor().newInstance()` is only called in `org.apache.spark.sql.execution.datasources.DataSource`. The behavior provides a good encapsulation.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

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