[GitHub] [spark] Ngone51 commented on a change in pull request #27686: [SPARK-30942] Fix the warning for requiring cores to be limiting resources

[GitBox <gi...@apache.org>]

Ngone51 commented on a change in pull request #27686: [SPARK-30942] Fix the warning for requiring cores to be limiting resources
URL: https://github.com/apache/spark/pull/27686#discussion_r383614812
 
 

 ##########
 File path: core/src/main/scala/org/apache/spark/SparkContext.scala
 ##########
 @@ -2798,7 +2798,7 @@ object SparkContext extends Logging {
         defaultProf.maxTasksPerExecutor(sc.conf) < cpuSlots) {
         throw new IllegalArgumentException("The number of slots on an executor has to be " +
           "limited by the number of cores, otherwise you waste resources and " +
-          "dynamic allocation doesn't work properly. Your configuration has " +
+          "some scheduling doesn't work properly. Your configuration has " +
 
 Review comment:
   Shall we give one or two examples for the "some scheduling" here?

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org