True I guess it's more than needed. Was using it as a little debugging but also thought people might want to know what their query ends up with.

Up to you to keep or trash

--
Mark Silverberg
marksilver.net/LinkedIn<http://marksilver.net/LinkedIn>
(Sent from my iPhone)

On Jul 10, 2013, at 7:47 PM, "Michael Wasser" <notifications@github.com<mailto:notifications@github.com>> wrote:


In lib/api/server.js:

>      result.rows.forEach(function (row) {
>        for(var k in row) {
>          if (row[k] === null) delete row[k];
>        }
>      });
> -    res.json(result.rows);
> +    res.json({
> +      meta: {
> +        rowCount: result.rowCount,
> +        sqlText: sqlQuery.text,
> +        sqlValues: sqlQuery.values,


Do you have a specific use case in mind in regard to transmitting the sql used? This seems more like a implementation detail that we would be exposing via API -- I doubt we would want people depending on a sql field like this in an API response. Thoughts?

-
Reply to this email directly or view it on GitHub<https://github.com/untoldone/bloomapi/pull/4/files#r5130705>.