Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/19464#discussion_r144319668
--- Diff: core/src/main/scala/org/apache/spark/rdd/HadoopRDD.scala ---
@@ -196,7 +196,11 @@ class HadoopRDD[K, V](
// add the credentials here as this can be called before SparkContext
initialized
SparkHadoopUtil.get.addCredentials(jobConf)
val inputFormat = getInputFormat(jobConf)
- val inputSplits = inputFormat.getSplits(jobConf, minPartitions)
+ val inputSplits = if
(sparkContext.getConf.get(FILTER_OUT_EMPTY_SPLIT)) {
--- End diff --
You can avoid duplicating `inputFormat.getSplits(jobConf, minPartitions)`
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
