Skip to content

Conversation

@hjgraca
Copy link
Contributor

@hjgraca hjgraca commented Dec 17, 2025

Please provide the issue number

Issue number: #1095

Summary

Changes

  • Add BatchProcessingStressTests for high-concurrency and sustained load validation
  • Add BatchProcessingThreadSafetyTests to verify thread-safe record processing
  • Add isolation tests for DynamoDB, Kinesis, SQS, and typed batch processors
  • Add StaticResultPropertyTests to validate static result property behavior
  • Add helper classes for concurrent invocation results, test event factory, and record handlers
  • Add project reference to BatchProcessing library in test project
  • Verify batch processor maintains correctness under high concurrency (20-100 concurrent invocations)
  • Validate thread safety with concurrent record processing and state isolation
  • Ensure no record leakage between concurrent invocations
  • Test stress scenarios with varying concurrency levels and iteration counts

User experience

Please share what the user experience looks like before and after this change

Checklist

Please leave checklist items unchecked if they do not apply to your change.

Is this a breaking change?

RFC issue number:

Checklist:

  • Migration process documented
  • Implement warnings (if it can live side by side)

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

- Add BatchProcessingStressTests for high-concurrency and sustained load validation
- Add BatchProcessingThreadSafetyTests to verify thread-safe record processing
- Add isolation tests for DynamoDB, Kinesis, SQS, and typed batch processors
- Add StaticResultPropertyTests to validate static result property behavior
- Add helper classes for concurrent invocation results, test event factory, and record handlers
- Add project reference to BatchProcessing library in test project
- Verify batch processor maintains correctness under high concurrency (20-100 concurrent invocations)
- Validate thread safety with concurrent record processing and state isolation
- Ensure no record leakage between concurrent invocations
- Test stress scenarios with varying concurrency levels and iteration counts
@boring-cyborg boring-cyborg bot added the tests label Dec 17, 2025
@pull-request-size pull-request-size bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Dec 17, 2025
@github-actions github-actions bot added the internal Maintenance changes label Dec 17, 2025
@sonarqubecloud
Copy link

@codecov
Copy link

codecov bot commented Dec 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.50%. Comparing base (eb7ba1d) to head (a3fe222).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1096      +/-   ##
===========================================
- Coverage    78.56%   78.50%   -0.07%     
===========================================
  Files          299      299              
  Lines        12375    12375              
  Branches      1492     1492              
===========================================
- Hits          9723     9715       -8     
- Misses        2178     2186       +8     
  Partials       474      474              

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

@hjgraca hjgraca merged commit 075be1e into develop Dec 18, 2025
8 of 9 checks passed
@hjgraca hjgraca deleted the feat(batch)-LMI-support branch December 18, 2025 12:37
@hjgraca hjgraca linked an issue Dec 18, 2025 that may be closed by this pull request
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Maintenance changes size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature request: Batch LMI support

2 participants