[chennaji9]

Which Issue(s) This PR Fixes

Fixes #11332

Brief Description

The blocking producer.send(message) call in the OpenMessaging SimpleProducer example printed Send async message OK, the exact line the real sendAsync(...) listener path prints below it, so the demo's output cannot distinguish its synchronous send from its asynchronous one. The sync block now prints Send sync message OK (one-line change).

How Did You Test This Change?

mvn -pl example -am -DskipTests compile (JDK 11, PR base commit) → BUILD SUCCESS with the change. Output-string change only; both labels verified against the example's actual control flow (blocking send vs sendAsync listener).

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.67%. Comparing base (9ea2ccd) to head (a594a07).

Additional details and impacted files

@@              Coverage Diff              @@
##             develop   #11344      +/-   ##
=============================================
- Coverage      49.68%   49.67%   -0.01%     
- Complexity     14331    14334       +3     
=============================================
  Files           1390     1390              
  Lines         103246   103246              
  Branches       13502    13502              
=============================================
- Hits           51298    51290       -8     
- Misses         45743    45754      +11     
+ Partials        6205     6202       -3     

☔ View full report in Codecov by Harness.
📢 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.

[RockteMQ-AI]

Review by github-manager-bot

Summary

Fixes the sync send label in SimpleProducer example to correctly identify the send type.

Findings

• [Info] Trivial one-line fix. Correct label improves example clarity.

---

Automated review by github-manager-bot

[RockteMQ-AI]

Issue Evaluation

Category: type/documentation | Status: Confirmed

The example output mislabeling is valid. The blocking producer.send(message) call prints "Send async message OK", which is identical to the sendAsync(...) listener output below it. This makes the example unable to distinguish synchronous from asynchronous sends.

Root Cause: Copy-paste error in the example's println statement.
Impact: Confusing for developers learning the OpenMessaging API; the example fails to demonstrate the difference between sync and async sends.
Severity: Low (example code only, no runtime impact)

The proposed one-line fix (changing the sync block's output to "Send sync message OK") correctly addresses the mislabeling.

---

Automated evaluation by github-manager-bot

[chennaji9]

Sorry to bother you @RongtongJin — just nudging this PR since it hasn't had any feedback yet. Tests and CI are green on my end, and the fix (label the sync send as sync in openmessaging SimpleProducer) is ready to go. Could you help review it and, if it works for you, merge it? More than happy to address any comments. Thanks!