Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add SQS module @SnsNotificationMessage batch messages support #1133

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

maxjiang153
Copy link

@maxjiang153 maxjiang153 commented Apr 6, 2024

馃摙 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

馃摐 Description

see: #1129

馃挕 Motivation and Context

馃挌 How did you test it?

馃摑 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • I updated reference documentation to reflect the change
  • All tests passing
  • No breaking changes

馃敭 Next steps

@github-actions github-actions bot added the component: sqs SQS integration related issue label Apr 6, 2024
@maxjiang153 maxjiang153 force-pushed the gh-1129 branch 3 times, most recently from 09ca31f to d1cdf70 Compare April 7, 2024 00:37
Copy link
Contributor

@tomazfernandes tomazfernandes left a comment

Choose a reason for hiding this comment

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

PR looks great @wmz7year, thanks!

Would you mind adding an entry in the docs for it?

Thanks!

@github-actions github-actions bot added the type: documentation Documentation or Samples related issue label Apr 15, 2024
@maxjiang153
Copy link
Author

maxjiang153 commented Apr 15, 2024

@tomazfernandes I've added the doc description for this, please check it out. And if everything is okay, I'll rebase it into a single commit.

Copy link
Contributor

@tomazfernandes tomazfernandes left a comment

Choose a reason for hiding this comment

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

Nice work with the docs @wmz7year! Just a couple of nits and we should be ready to go.


The @SnsNotificationMessage annotation can be used for both single and batch message processing.

For single message processing, you can use the @SnsNotificationMessage annotation as follows:
Copy link
Contributor

Choose a reason for hiding this comment

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

We avoid using you and and use a more formal tone instead.

So maybe something like the @SnsNotificationMessage annotation can be used as follows:?

Copy link
Author

Choose a reason for hiding this comment

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

done

@@ -626,6 +631,16 @@ public void listen(@SnsNotificationMessage Pojo pojo) {
}
----

For batch message processing, you can use the @SnsNotificationMessage annotation with a List<Pojo> parameter:
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here about using you

Copy link
Author

Choose a reason for hiding this comment

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

done, check it out again plz.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: sqs SQS integration related issue type: documentation Documentation or Samples related issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants