influxdata / influxdata/influxdb

200 OK returned when max-concurrent-queries limit exceeded

Open
#14,654 4 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Rust
Stars
31.7k
Forks
3.7k
Avg merge
13h 37m
Merged PRs (30d)
8

Description

Does it make more sense if influxdb returns 503?

```
curl -v http://localhost:8086/query?q=show%20databases [18/90225]
* About to connect() to localhost port 8086 (#0)
* Trying 127.0.0.1...
* Connected to localhost (127.0.0.1) port 8086 (#0)
> GET /query?q=show%20databases HTTP/1.1
> User-Agent: curl/7.29.0
> Host: localhost:8086
> Accept: */*
>
< HTTP/1.1 200 OK
< Content-Type: application/json
< Request-Id: c75ec0d4-be90-11e9-acf2-448a5bdc9e68
< X-Influxdb-Build: OSS
< X-Influxdb-Version: 1.7.7rc1
< X-Request-Id: c75ec0d4-be90-11e9-acf2-448a5bdc9e68
< Date: Wed, 14 Aug 2019 12:41:04 GMT
< Transfer-Encoding: chunked
<
{"results":[{"statement_id":0,"error":"max-concurrent-queries limit exceeded(2, 2)"}]}
```

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing the request against the /query endpoint while max-concurrent-queries is exceeded, using the response shown in the issue as the baseline. Trace where the limit error is converted into the HTTP response; done means the same condition returns 503 instead of 200 while still reporting the limit error.

Written by the indexing model from the issue text.

Assessment

Domain
api, databases
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.