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.
Hm, we should explore this a bit more.  Is the problem with scala-metrics
or the base dropwizard metrics?  Registry usually finds an existing
instance rather than trying to re-register.

Also, we just have one instance per final class.    I think the solution is
to have one metric for the Spatial cache and one for the Map cache, as long
as its clear the metric name has that in it.  You can override the default
metric name easily.

On Tue, Nov 11, 2014 at 4:45 PM, 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
>
> Maybe the strictly correct thing to do here is to scope all of the metrics
> per cache instance? But right now we don't instantiate more than one of
> each type of cache in the service - only in the unit tests.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/socrata/geospace/pull/28/files#r20193818>.
>



-- 
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)