pinot icon indicating copy to clipboard operation
pinot copied to clipboard

[multi-stage] Query timeout overwrite

Open 61yao opened this issue 3 years ago • 2 comments

Currently, the query option time doesn't overwrite correctly.

We should have:

user-config override should always be considered as the total time (maybe minus planning time, i will explain later)

with this in mind, all the current timeout configuration during query should also be considered as total time all stages have a fixed timeout at an absolutely ts value = sys.currentTimeNano() + desiredTimeoutNano

if any stage hits this timeout, they will all agree to terminate.

See details in: https://github.com/apache/pinot/pull/9571#issuecomment-1278094979

61yao avatar Oct 13 '22 20:10 61yao

Is this for multi-stage (v2) engine or also apply to v1? I believe v1 has the correct behavior.

Jackie-Jiang avatar Oct 15 '22 21:10 Jackie-Jiang

This is for multi-stage only. Let me update the issue title

61yao avatar Oct 19 '22 23:10 61yao