chennaji9 opened a new pull request, #11298:
URL: https://github.com/apache/rocketmq/pull/11298

   <!-- Please make sure the target branch is right. In most case, the target 
branch should be `develop`. -->
   
   ### Which Issue(s) This PR Fixes
   
   Fixes #11291
   
   ### Brief Description
   
   The transactional producer snippet in `docs/en/Example_Transaction.md` 
carried a push-consumer import list (`DefaultMQPushConsumer`, 
`MessageListenerConcurrently`, `MessageExt`, ...) while its body uses 
`TransactionMQProducer`, `TransactionListener`, `ThreadPoolExecutor`, 
`ExecutorService`, `ThreadFactory`, `Message`, `SendResult` and 
`RemotingHelper` — so the block failed with `cannot find symbol: 
TransactionListener`.
   
   - Section 2.1 now imports exactly what its body uses (mirroring 
`example/src/main/java/org/apache/rocketmq/example/transaction/TransactionProducer.java`).
   - Section 2.2's `TransactionListenerImpl` block gains its missing imports 
(`AtomicInteger`, `ConcurrentHashMap`, `LocalTransactionState`, 
`TransactionListener`, `Message`, `MessageExt`).
   
   ### How Did You Test This Change?
   
   Extracted both java blocks from the page and compiled them together with 
`javac` against the repo's `client`/`common`/`remoting` module classes: the 
updated snippets compile cleanly (exit 0); before the change the producer block 
failed on the first missing symbol.
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to