Skip to content

fix(tests): increase shutdown deadlines from 200ms to 1s in parquet adaptive_schema tests#2812

Merged
drewrelmas merged 2 commits intoopen-telemetry:mainfrom
AaronRM:fix-test_adaptive_schema_optional_columns
May 4, 2026
Merged

fix(tests): increase shutdown deadlines from 200ms to 1s in parquet adaptive_schema tests#2812
drewrelmas merged 2 commits intoopen-telemetry:mainfrom
AaronRM:fix-test_adaptive_schema_optional_columns

Conversation

@AaronRM
Copy link
Copy Markdown
Contributor

@AaronRM AaronRM commented May 2, 2026

Change Summary

Increase shutdown timeouts in parquet adaptive_schema tests to address test flakiness on slow CI runs.

What issue does this PR close?

How are these changes tested?

  • Verified tests pass locally

Are there any user-facing changes?

No. This is a test-only change.

@AaronRM AaronRM requested a review from a team as a code owner May 2, 2026 15:25
@github-actions github-actions Bot added the rust Pull requests that update Rust code label May 2, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.03%. Comparing base (9aa767e) to head (8f53ce2).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2812      +/-   ##
==========================================
- Coverage   86.03%   86.03%   -0.01%     
==========================================
  Files         704      704              
  Lines      264591   264588       -3     
==========================================
- Hits       227636   227633       -3     
  Misses      36431    36431              
  Partials      524      524              
Components Coverage Δ
otap-dataflow 86.97% <100.00%> (-0.01%) ⬇️
query_abstraction 80.61% <ø> (ø)
query_engine 90.76% <ø> (ø)
otel-arrow-go 52.45% <ø> (ø)
quiver 92.25% <ø> (ø)
🚀 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.

@drewrelmas drewrelmas enabled auto-merge May 4, 2026 16:09
@drewrelmas drewrelmas added this pull request to the merge queue May 4, 2026
Merged via the queue into open-telemetry:main with commit 00b00e3 May 4, 2026
78 of 79 checks passed
@AaronRM AaronRM deleted the fix-test_adaptive_schema_optional_columns branch May 4, 2026 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rust Pull requests that update Rust code

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants