Thanks! Just fixed!

Kevin


On Mon, Jun 3, 2013 at 2:41 PM, Ryan LeCompte <notifications@github.com>wrote:

> In scalding-core/src/main/scala/com/twitter/scalding/Mode.scala:
>
> >    implicit var mode : Mode = Local(false)
> >
> > +  // Map the specific mode to Job's UUID
> > +  def putMode(mode : Mode, args : Args) = synchronized {
> > +    modeMap.put(args("scalding.job.mode"), mode)
> > +  }
> > +
> > +  // Get the specific mode by UUID
> > +  def getMode(args : Args) = synchronized {
> > +    modeMap.get(args("scalding.job.mode")).getOrElse(mode)
>
> This could also be:
>
>   modeMap.getOrElse(args("scalding.job.mode"), mode)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/twitter/scalding/pull/448/files#r4510919>
> .
>



-- 
Kevin Lin | Twitter, Inc.
1355 Market St. | San Francisco, CA | 94103

Follow me: @reconditesea <https://twitter.com/reconditesea>