HeartSaVioR commented on a change in pull request #26162: [SPARK-29438][SS] Use 
partition ID of StateStoreAwareZipPartitionsRDD for determining partition ID of 
state store in stream-stream join
URL: https://github.com/apache/spark/pull/26162#discussion_r336830809
 
 

 ##########
 File path: 
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamingSymmetricHashJoinHelper.scala
 ##########
 @@ -207,13 +206,13 @@ object StreamingSymmetricHashJoinHelper extends Logging {
    */
   class StateStoreAwareZipPartitionsRDD[A: ClassTag, B: ClassTag, V: ClassTag](
       sc: SparkContext,
-      f: (Iterator[A], Iterator[B]) => Iterator[V],
-      rdd1: RDD[A],
-      rdd2: RDD[B],
+      var f: (Int, Iterator[A], Int, Iterator[B]) => Iterator[V],
+      var rdd1: RDD[A],
+      var rdd2: RDD[B],
       stateInfo: StatefulOperatorStateInfo,
       stateStoreNames: Seq[String],
       @transient private val storeCoordinator: 
Option[StateStoreCoordinatorRef])
-      extends ZippedPartitionsRDD2[A, B, V](sc, f, rdd1, rdd2) {
 
 Review comment:
   Actually the description still hold true, as it only changes the function 
signature from ZippedPartitionsRDD2 (but cannot extend ZippedPartitionsRDD2 due 
to the change).

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to