prev is fine here because it is not a field. Only a ctor arg. 

> On Feb 12, 2014, at 1:47 AM, Mridul Muralidharan <notifications@github.com> wrote:
> 
> In core/src/main/scala/org/apache/spark/rdd/ZippedWithIndexRDD.scala:
> 
> > +class ZippedWithIndexRDDPartition(val prev: Partition, val startIndex: Long)
> > +  extends Partition with Serializable {
> > +  override val index: Int = prev.index
> > +}
> > +
> > +/**
> > + * Represents a RDD zipped with its element indices. The ordering is first based on the partition
> > + * index and then the ordering of items within each partition. So the first item in the first
> > + * partition gets index 0, and the last item in the last partition receives the largest index.
> > + *
> > + * @param prev parent RDD
> > + * @tparam T parent RDD item type
> > + */
> > +private[spark]
> > +class ZippedWithIndexRDD[T: ClassTag](prev: RDD[T]) extends RDD[(T, Long)](prev) {
> > +
> prev should be transient ?
> 
> —
> Reply to this email directly or view it on GitHub.