Fix BrokenPipeError handling in rdump

[yunzheng]

Fixes #186

[codecov]

Codecov Report

❌ Patch coverage is 71.42857% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.38%. Comparing base (f2606d1) to head (3964b57).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
flow/record/tools/rdump.py	71.42%	6 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #188      +/-   ##
==========================================
+ Coverage   83.32%   83.38%   +0.05%     
==========================================
  Files          35       35              
  Lines        3713     3707       -6     
==========================================
- Hits         3094     3091       -3     
+ Misses        619      616       -3     

Flag	Coverage Δ
unittests	83.38% <71.42%> (+0.05%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[yunzheng]

@JSCU-CNI can you also check if this PR works for you regarding Elastic search exceptions? as I basically reverted the RecordWriter back to a with statement that was removed in this commit: c482853

[JSCU-CNI]

@JSCU-CNI can you also check if this PR works for you regarding Elastic search exceptions? as I basically reverted the RecordWriter back to a with statement that was removed in this commit: c482853

I need more time to test this, but I think the existing elastic tests should catch any breaking changes.

[yunzheng]

@JSCU-CNI can you also check if this PR works for you regarding Elastic search exceptions? as I basically reverted the RecordWriter back to a with statement that was removed in this commit: c482853

I need more time to test this, but I think the existing elastic tests should catch any breaking changes.

i'll go ahead and merge this, if any issues pop up, please raise a new issue.

out of date