[GitHub] [spark] srowen commented on pull request #35696: [SPARK-38361][SQL] Factory method getConnection should take Partition as optional parameter.

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

srowen commented on pull request #35696:
URL: https://github.com/apache/spark/pull/35696#issuecomment-1055495878


   This isn't used in Spark though ... what's the use case? isn't your 'database' just a shard of a database in this case, so would be specified in a connection string or something?


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