Skip to content

feat(core): add external configuration for lag calculation window size#3025

Merged
ideoma merged 2 commits intomasterfrom
vi_set_lag_windows_size
Feb 27, 2023
Merged

feat(core): add external configuration for lag calculation window size#3025
ideoma merged 2 commits intomasterfrom
vi_set_lag_windows_size

Conversation

@bluestreak01
Copy link
Member

Ability to set size of the window, which is used by O3 automatic lag algo to maintain correct lag size. Before this lag window was hardcoded to have 4 entries, which may be insufficient for some ingestion data patterns.

The new option can be set via:

cairo.o3.lag.calculation.windows.size=128

Copy link
Contributor

@marregui marregui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ideoma
Copy link
Collaborator

ideoma commented Feb 27, 2023

[PR Coverage check]

😍 pass : 6 / 6 (100.00%)

file detail

path covered line new line coverage
🔵 io/questdb/PropServerConfiguration.java 2 2 100.00%
🔵 io/questdb/cairo/DefaultCairoConfiguration.java 1 1 100.00%
🔵 io/questdb/PropertyKey.java 1 1 100.00%
🔵 io/questdb/cairo/TableWriter.java 2 2 100.00%

@ideoma ideoma merged commit 8b7f1da into master Feb 27, 2023
@ideoma ideoma deleted the vi_set_lag_windows_size branch February 27, 2023 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants