Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/2485#discussion_r18049333
--- Diff:
yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ClientBase.scala ---
@@ -64,14 +64,18 @@ private[spark] trait ClientBase extends Logging {
s"memory capability of the cluster ($maxMem MB per container)")
val executorMem = args.executorMemory + executorMemoryOverhead
if (executorMem > maxMem) {
- throw new IllegalArgumentException(s"Required executor memory
($executorMem MB) " +
+ throw new IllegalArgumentException(s"Required executor memory
($args.executorMemmory+$executorMemoryOverhead MB) " +
s"is above the max threshold ($maxMem MB) of this cluster!")
}
val amMem = args.amMemory + amMemoryOverhead
if (amMem > maxMem) {
- throw new IllegalArgumentException(s"Required AM memory ($amMem MB)
" +
+ throw new IllegalArgumentException(s"Required AM memory
($args.amMemory+$amMemoryOverhead MB) " +
--- End diff --
Also this line is too long (100 chars max)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
