pivot icon indicating copy to clipboard operation
pivot copied to clipboard

Query Timeout is Overwritten With Hardcoded Ajax Params

Open alodium opened this issue 8 years ago • 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.

alodium avatar Jul 28 '16 06:07 alodium