Skip to content
This repository has been archived by the owner on Jan 20, 2022. It is now read-only.

ADd a means to set defaults outside the planner #351

Merged
merged 1 commit into from
Nov 9, 2013
Merged

ADd a means to set defaults outside the planner #351

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions summingbird-scalding/src/main/scala/com/twitter/summingbird/scalding/ScaldingPlatform.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -232,10 +232,10 @@ object Scalding {

private def getOrElse[T: Manifest](options: Map[String, Options], idOpt: Option[String], default: T): T =
(for {
id <- idOpt
id <- idOpt.map(List(_, "DEFAULT")).getOrElse(List("DEFAULT"))
innerOpts <- options.get(id)
option <- innerOpts.get[T]
} yield option).getOrElse(default)
} yield option).headOption.getOrElse(default)

@annotation.tailrec
private def getFirst[T: Manifest](options: Map[String, Options], names: List[String]): Option[T] =
Expand Down