implydata/pivot

Query Timeout is Overwritten With Hardcoded Ajax Params

alodium opened this issue · 0 comments

Hi.

When setting a query timeout in the config file (in "clusters" section), it is expected that this timeout will apply to the whole roundabout, from the browser to druid and back.

The actual result is that while the db timeout is as configured in the settings file, the ajax request will fail after one minute, this is because a hardcoded param in file: node_modules/qajax/src/qajax.js line 118.

To fix this, one can import the timeout param in file common/models/cluster/cluster.ts line 270 with the adding of timeout param to the clusters object:
timeout: this.timeout

    return new Cluster({
      name: this.name,
      type: this.type
      type: this.type,
      timeout: this.timeout
    })

And then add it to file src/client/utils/ajax/ajax.ts line 43 with something like

if (this.cluster.timeout) ajaxObject.timeout = this.cluster.timeout;

Hope the description is helpful,

Lior.