Skip to content

Conversation

SungJin1212
Copy link
Member

@SungJin1212 SungJin1212 commented Mar 24, 2025

This PR makes out-of-order ingestion for native histograms automatically enabled when -ingester.out-of-order-time-window > 0 and -blocks-storage.tsdb.enable-native-histograms=true as the Prometheus changed its behavior at the prometheus/prometheus#16207.

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@SungJin1212 SungJin1212 force-pushed the Make-ooo-NH-ingestion-always-enable branch from 11847fa to 31929d3 Compare March 24, 2025 07:04
@SungJin1212 SungJin1212 changed the title Make ooo native ingestion automatically enabled Make ooo ingestion for native ingestion automatically enabled Mar 24, 2025
@SungJin1212 SungJin1212 force-pushed the Make-ooo-NH-ingestion-always-enable branch from 31929d3 to 4744986 Compare March 24, 2025 07:46
@pull-request-size pull-request-size bot added size/L and removed size/M labels Mar 24, 2025
@SungJin1212 SungJin1212 force-pushed the Make-ooo-NH-ingestion-always-enable branch 3 times, most recently from bf771c3 to f75d87e Compare March 24, 2025 08:03
@SungJin1212 SungJin1212 changed the title Make ooo ingestion for native ingestion automatically enabled Make ooo native histograms ingestion automatically enabled Mar 24, 2025
@SungJin1212
Copy link
Member Author

How about enabling NH ingestion by default to expand the user pool to use it?

Copy link
Member

@friedrichg friedrichg left a comment

Choose a reason for hiding this comment

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

This simplifies the config, Thanks!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Mar 24, 2025
@SungJin1212 SungJin1212 force-pushed the Make-ooo-NH-ingestion-always-enable branch from f75d87e to ab079eb Compare March 25, 2025 06:30
@SungJin1212 SungJin1212 force-pushed the Make-ooo-NH-ingestion-always-enable branch from ab079eb to 53320c5 Compare March 25, 2025 06:31
@SungJin1212 SungJin1212 force-pushed the Make-ooo-NH-ingestion-always-enable branch from 53320c5 to 370988b Compare March 25, 2025 06:51
Copy link
Member

@CharlieTLe CharlieTLe left a comment

Choose a reason for hiding this comment

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

Thanks! Can we mark this in the CHANGELOG as a CHANGE instead of a FEATURE? The removal of the flag/field may cause some installations to break if they've set it before previously.

@yeya24
Copy link
Contributor

yeya24 commented Mar 25, 2025

@CharlieTLe there is no flag being removed as the flag was added recently and has not been released.

Copy link
Contributor

@yeya24 yeya24 left a comment

Choose a reason for hiding this comment

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

Thanks

@yeya24 yeya24 merged commit be0fc7f into cortexproject:master Mar 25, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/ingester lgtm This PR has been approved by a maintainer size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants