tgravescs commented on a change in pull request #27367: [SPARK-30638][CORE] Add 
resources allocated to PluginContext
URL: https://github.com/apache/spark/pull/27367#discussion_r373292752
 
 

 ##########
 File path: 
core/src/main/scala/org/apache/spark/resource/ResourceProfileBuilder.scala
 ##########
 @@ -31,7 +31,7 @@ import org.apache.spark.annotation.Evolving
  * requirements between stages.
  */
 @Evolving
-class ResourceProfileBuilder() {
+private [spark] class ResourceProfileBuilder() {
 
 Review comment:
   the intention was to be private until rest of stage level scheduling done 
and just remove private, so would rather not remove evolving, but you are right 
this really should be separated out as its not necessarily related, I just 
noticed it missing, I can separate it out. 

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