influxdata / influxdata/influxdb

reasonable MaxMemoryBytesPerQuery

Open
#17,376 0 comments 0 reactions 0 assignees View on GitHub
kind/feature-request team/compute
Dominant language
Rust
Stars
31.7k
Forks
3.7k
Avg merge
13h 37m
Merged PRs (30d)
8

Description

https://github.com/influxdata/influxdb/blob/f5e854748228ca3115e8be2dcaade764bc9c5264/cmd/influxd/launcher/launcher.go#L592

This value should be settable via flag or env variable. This is to prevent a bad task or query from OOMing an instance.

Contributor guide

Open the contributing guide

Research direction

Start in cmd/influxd/launcher/launcher.go at the referenced MaxMemoryBytesPerQuery setting, then inspect nearby launcher configuration for existing flag or environment-variable patterns. The work is done when the memory limit can be configured through one of those mechanisms and protects the instance from a task or query causing an OOM.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
databases
Issue type
Feature
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.