Introduce default rows limit in addition to max rows.  Default is 1k.

The only exception you get from config.get* is for missing config.

On Mon, Sep 15, 2014 at 9:34 AM, rjmac <notifications@github.com> wrote:

> In
> src/main/scala/com/socrata/querycoordinator/QueryCoordinatorConfig.scala:
>
> > @@ -18,6 +18,7 @@ class QueryCoordinatorConfig(config: Config, root: String) {
> >    val schemaTimeout = config.getMilliseconds(k("get-schema-timeout")).longValue.millis
> >    val queryTimeout = config.getMilliseconds(k("query-timeout")).longValue.millis
> >    val maxRows = try { Some(config.getInt(k("max-rows"))) } catch { case _: ConfigException.Missing => None}
>
> Never use Try unless you really do intend to catch all exceptions. If
> this used the socrata-thirdpart-utils config stuff, it could be
> optionally(getInt("max-rows"))
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/socrata/query-coordinator/pull/13/files#r17552665>.
>



-- 
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)