Skip to content

Commit

Permalink
[SPARK-3848] yarn alpha doesn't build on master
Browse files Browse the repository at this point in the history
yarn alpha build was broken by #2432
as it added an argument to YarnAllocator but not to yarn/alpha YarnAllocationHandler
commit 79e45c9

Author: Kousuke Saruta <[email protected]>

Closes #2715 from sarutak/SPARK-3848 and squashes the following commits:

bafb8d1 [Kousuke Saruta] Fixed parameters for the default constructor of alpha/YarnAllocatorHandler.
  • Loading branch information
sarutak authored and tgravescs committed Oct 8, 2014
1 parent 7fca8f4 commit f18dd59
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ private[yarn] class YarnAllocationHandler(
args: ApplicationMasterArguments,
preferredNodes: collection.Map[String, collection.Set[SplitInfo]],
securityMgr: SecurityManager)
extends YarnAllocator(conf, sparkConf, args, preferredNodes, securityMgr) {
extends YarnAllocator(conf, sparkConf, appAttemptId, args, preferredNodes, securityMgr) {

private val lastResponseId = new AtomicInteger()
private val releaseList: CopyOnWriteArrayList[ContainerId] = new CopyOnWriteArrayList()
Expand Down

0 comments on commit f18dd59

Please sign in to comment.