Skip to content

Conversation

@forshev
Copy link
Contributor

@forshev forshev commented Nov 7, 2025

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes #249


  • I have read and followed all requirements in CONTRIBUTING.md;
  • I used LLM/AI assistance to make this pull request;

If you have used LLM/AI assistance please provide model name and full prompt:

Model: {{model-name}}
Prompt: {{prompt}}

@codecov-commenter
Copy link

codecov-commenter commented Nov 7, 2025

Codecov Report

❌ Patch coverage is 28.12500% with 23 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.88%. Comparing base (36016e6) to head (5573b1e).

Files with missing lines Patch % Lines
proxy/search/ingestor.go 0.00% 9 Missing and 3 partials ⚠️
storeapi/grpc_v1.go 0.00% 3 Missing and 3 partials ⚠️
frac/processor/eval_tree.go 55.55% 4 Missing ⚠️
frac/processor/search.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #250   +/-   ##
=======================================
  Coverage   71.88%   71.88%           
=======================================
  Files         199      199           
  Lines       14570    14585   +15     
=======================================
+ Hits        10473    10484   +11     
- Misses       3364     3366    +2     
- Partials      733      735    +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@forshev forshev force-pushed the 249-agg-limits-error-messages branch from 618ca5f to eaed919 Compare November 7, 2025 09:16
@eguguchkin eguguchkin added this to the v0.62.5 milestone Nov 7, 2025
@forshev forshev force-pushed the 249-agg-limits-error-messages branch from eaed919 to 5573b1e Compare November 7, 2025 17:43
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.

Return clear error messages when aggregation limit exceeded

4 participants