Skip to content

Added option to stream directory items#203

Merged
GlassOfWhiskey merged 1 commit intomasterfrom
dirent-streaming
Feb 3, 2026
Merged

Added option to stream directory items#203
GlassOfWhiskey merged 1 commit intomasterfrom
dirent-streaming

Conversation

@marcoSanti
Copy link
Member

@marcoSanti marcoSanti commented Feb 3, 2026

This commit adds the option to stream the directory entities as described in the CAPIO-CL specification by spawning a new thread waiting for a new entry in the directory's CapioFile object.

It also fixes a bug on a condition variable within the CapioFile class, which previously allowed threads waiting for data to continue only as soon as the CapioFile was committed, instead of when data was actually available.

This commit adds the option to stream the directory entities as
described in the CAPIO-CL specification. It does this by spawning a new
thread waiting for a new entry in the directory's CapioFile.

It also fixes a bug on the condition of a condition variable within the
CapioFile class, that previusly allowed threads waiting for data to
continue only as soon as the CapioFile was committed, instead of when
data was actually available.
@codecov
Copy link

codecov bot commented Feb 3, 2026

Codecov Report

❌ Patch coverage is 23.52941% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 56.06%. Comparing base (2a0a9d4) to head (514d721).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
capio/server/include/utils/common.hpp 25.00% 3 Missing and 9 partials ⚠️
capio/server/include/utils/capio_file.hpp 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #203      +/-   ##
==========================================
- Coverage   56.34%   56.06%   -0.29%     
==========================================
  Files          71       71              
  Lines        3468     3480      +12     
  Branches     2151     2162      +11     
==========================================
- Hits         1954     1951       -3     
+ Misses        668      590      -78     
- Partials      846      939      +93     

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

@GlassOfWhiskey GlassOfWhiskey merged commit 05f7131 into master Feb 3, 2026
30 checks passed
@GlassOfWhiskey GlassOfWhiskey deleted the dirent-streaming branch February 3, 2026 17:28
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.

2 participants