Skip to content

Conversation

@nrc
Copy link
Contributor

@nrc nrc commented Nov 18, 2025

Previously, ObjectStoreIoSource always used the default value, even if it was overridden using ObjectStoreReadSource::with_concurrency.

Sorry, I have no idea how to nicely test this.

…Source

Signed-off-by: Nick Cameron <nrc@ncameron.org>
Copy link
Contributor

@a10y a10y left a comment

Choose a reason for hiding this comment

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

thank you!

@a10y a10y added the changelog/fix A bug fix label Nov 18, 2025
@a10y a10y enabled auto-merge (squash) November 18, 2025 02:16
@codecov
Copy link

codecov bot commented Nov 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.28%. Comparing base (8f7b707) to head (141df7d).
⚠️ Report is 1 commits behind head on develop.

☔ 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.

@a10y a10y merged commit b1054d0 into vortex-data:develop Nov 18, 2025
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/fix A bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants