Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change the policy of the queue(DataCarrier) L1 in the metric aggregate worker to IF_POSSIBLE mode #11554

Merged
merged 4 commits into from
Nov 16, 2023

Conversation

wu-sheng
Copy link
Member

  • If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
  • Update the CHANGES log.
  • Change the DataCarrier IF_POSSIBLE strategy to use ArrayBlockingQueue implementation.
  • Change the policy of the queue(DataCarrier) L1 metric aggregate worker to IF_POSSIBLE mode.
  • Add self-observability metric metrics_aggregator_abandon to count the number of abandon metrics.

…ue implementation.

* Change the policy of the queue(DataCarrier) L1 metric aggregate worker to IF_POSSIBLE mode.
* Add self-observability metric `metrics_aggregator_abandon` to count the number of abandon metrics.
@wu-sheng wu-sheng added core feature Core and important feature. Sometimes, break backwards compatibility. backend OAP backend related. metrics Metrics and meter labels Nov 16, 2023
@wu-sheng wu-sheng added this to the 9.7.0 milestone Nov 16, 2023
@wu-sheng wu-sheng requested a review from kezhenxu94 November 16, 2023 02:01
@wu-sheng
Copy link
Member Author

@kezhenxu94 We need to update self-observability accordingly.

@wu-sheng wu-sheng changed the title Change the policy of the queue(DataCarrier) L1 metric aggregate worker to IF_POSSIBLE mode Change the policy of the queue(DataCarrier) L1 in the metric aggregate worker to IF_POSSIBLE mode Nov 16, 2023
Co-authored-by: kezhenxu94 <kezhenxu94@apache.org>
@wu-sheng wu-sheng merged commit 257684a into master Nov 16, 2023
166 checks passed
@wu-sheng wu-sheng deleted the abandon branch November 16, 2023 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend OAP backend related. core feature Core and important feature. Sometimes, break backwards compatibility. metrics Metrics and meter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants