Skip to content

docs(batch): new visuals and error handling section #2857

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

heitorlessa
Copy link
Contributor

@heitorlessa heitorlessa commented Jul 27, 2023

Issue number: #2848

⭐ Inspired by @94Sip and @duc00 recent issues on Batch docs - THANK YOU

Summary

Revamps Batch Processing documentation to include visuals (state, class, sequences), several code annotations for more contextual information, improved Accessing processed messages, and new Error handling section.

Changes

Please provide a summary of what's being changed

  • Diagram to introduce Batch Processing feature
  • Diagram to introduce behavior without Batch Processing feature
  • Cleaned up getting started
  • Added code annotations for key examples for more contextual information (e.g., any exception is caught, etc.)
  • New Error handling section
  • Extend Partial failure mechanics section with sequence diagrams
  • Sequence diagrams to explain behavior for each Processor and failure modes
  • Improved callouts in Processing messages asynchronously
  • Added sample raw processed messages in Accessing processed messages
  • Improved wording to ease reading 3 tuple items in Accessing processed messages

User experience

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

Intro diagram

image

🆕 Error handling section

image

Code annotation

image

Code annotation

image

🆕 Sequence diagrams section

SQS Standard queues

image

SQS FIFO

image

Kinesis and DynamoDB

image image

Raw processing revamped plus samples

image

Class diagram for Bring Your Own Processor

image

Checklist

If your change doesn't seem to apply, please leave them unchecked.

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.

@heitorlessa heitorlessa requested a review from a team July 27, 2023 09:37
@boring-cyborg boring-cyborg bot added the documentation Improvements or additions to documentation label Jul 27, 2023
@pull-request-size pull-request-size bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jul 27, 2023
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants