tgravescs commented on a change in pull request #27773: [SPARK-29154][CORE] 
Update Spark scheduler for stage level scheduling
URL: https://github.com/apache/spark/pull/27773#discussion_r390306835
 
 

 ##########
 File path: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala
 ##########
 @@ -529,6 +549,28 @@ private[spark] class DAGScheduler(
     parents
   }
 
+  private[scheduler] def getResourceProfilesForRDDsInStage(
 
 Review comment:
   I'm not sure what you mean by this, ResourceProfiles don't track RDDs, RDDs 
track what ResourceProfile they have. The best you could do is check to see if 
there is more then 1 Resource Profile (the default one).  The only way to know 
if an RDD has the profile is to look at them.  I'm looking to see if I can 
combine, it doesn't look as messy now, but let me finish codig it

----------------------------------------------------------------
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