Ability to georegion code based on a string

This enables directly georegion coding against a column containing zip code strings (eg. "98122") instead of having to geocode the zip code to an additional point column and then georegion coding that column against the region polygon.
Urmila, maybe what we can do is write our own gauge wrapper for now, and
contribute the fix back to scala-metrics.... what do you think?   Though if
the tests are fine we can leave it....

On Wed, Nov 12, 2014 at 11:27 AM, urmilan <notifications@github.com> wrote:

> In src/main/scala/com/socrata/geospace/regioncache/RegionCache.scala:
>
> >
> > -  val cacheSizeGauge = metrics.gauge("num-entries") { cache.size }
> > +  // There's a bug in the scala-metrics library - metrics.gauge doesn't check if
> > +  // the metric already exists before trying to registering it again.
> > +  // Because of this, unit tests fail unless we give the gauge metric a unique scope per instance
>
> s/codahale/dropwizard maybe... I am confused by things inside
> io.dropwizard.metrics having a namespace of com.codahale.metrics.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/socrata/geospace/pull/28/files#r20243528>.
>



-- 
Evan Chan   |   Principal Engineer, Socrata     |    @evanfchan
*Socrata <http://www.socrata.com/> *| The Data Experience Company™
*Socrata isn't about worldly success, but about changing the world
successfully. This is why I'm excited to be here.*
*What does Socrata do? <https://www.youtube.com/watch?v=yyz86MlkriQ> *(watch
our 2 min video)