Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10562#discussion_r48996433
--- Diff: core/src/main/scala/org/apache/spark/rdd/RDD.scala ---
@@ -1291,11 +1291,11 @@ abstract class RDD[T: ClassTag](
} else {
val buf = new ArrayBuffer[T]
val totalParts = this.partitions.length
- var partsScanned = 0
+ var partsScanned = 0L
--- End diff --
Fair point, in practice this all but certainly won't happen. Note that this
patch was already committed to `master` making this a `Long`. It doesn't hurt
and is very very theoretically more correct locally. I suppose I don't think
it's worth updating again, but I do not feel strongly about it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
