apache / apache/pinot

[multi-stage] Query timeout overwrite

Open
#9,589 2 comments 0 reactions 1 assignee Claimed by @61yao View on GitHub
multi-stage
Dominant language
Java
Stars
6.1k
Forks
1.5k
Avg merge
1d 21h
Merged PRs (30d)
189

Description

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

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.