diff --git a/lib/cartodb/backends/dataview.js b/lib/cartodb/backends/dataview.js index 4f50121f..99a115e6 100644 --- a/lib/cartodb/backends/dataview.js +++ b/lib/cartodb/backends/dataview.js @@ -59,17 +59,11 @@ DataviewBackend.prototype.getDataview = function (mapConfigProvider, user, param return callback(error); } - dataview.getResult(pg, overrideParams, function (err, dataviewResult) { + dataview.getResult(pg, overrideParams, function (err, dataviewResult, stats = {}) { if (err) { return callback(err); } - const stats = {}; - - if (dataviewResult && dataviewResult.usesOverviews) { - stats.usesOverviews = dataviewResult.usesOverviews; - } - return callback(null, dataviewResult, stats); }); }); diff --git a/lib/cartodb/models/dataview/base.js b/lib/cartodb/models/dataview/base.js index 13743f5b..366a458c 100644 --- a/lib/cartodb/models/dataview/base.js +++ b/lib/cartodb/models/dataview/base.js @@ -21,7 +21,7 @@ function getPGTypeName (pgType) { module.exports = class BaseDataview { getResult (psql, override, callback) { - this.sql(psql, override, (err, query, flags = { usesOverviews : false }) => { + this.sql(psql, override, (err, query, flags = null) => { if (err) { return callback(err); } @@ -33,10 +33,15 @@ module.exports = class BaseDataview { result = this.format(result, override); result.type = this.getType(); - result.usesOverviews = flags.usesOverviews; - - return callback(null, result); + //Overviews looging + const stats = {}; + + if (flags && flags.usesOverviews) { + stats.usesOverviews = true; + } + + return callback(null, result, stats); }, true); // use read-only transaction }); }