cloud-fan commented on code in PR #37937: URL: https://github.com/apache/spark/pull/37937#discussion_r975028160
########## core/src/main/scala/org/apache/spark/rdd/JdbcRDD.scala: ########## @@ -32,7 +32,6 @@ private[spark] class JdbcPartition(idx: Int, val lower: Long, val upper: Long) e override def index: Int = idx } -// TODO: Expose a jdbcRDD function in SparkContext and mark this as semi-private Review Comment: This is a very old TODO and we need to revisit if this is still necessary. Since Spark SQL is the new core, I'm not sure if anyone is interested in a new API to create jdbc RDD. -- 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-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
