gnodet-bot commented on code in PR #28135:
URL: https://github.com/apache/camel/pull/28135#discussion_r4322575852
##########
components/camel-azure/camel-azure-eventhubs/src/main/java/org/apache/camel/component/azure/eventhubs/EventHubsConsumer.java:
##########
@@ -222,7 +222,7 @@ private void onErrorListener(final ErrorContext
errorContext) {
*
* @param exchange the exchange
*/
- private void processCommit(final Exchange exchange, final EventContext
eventContext) {
+ void processCommit(final Exchange exchange, final EventContext
eventContext) {
Review Comment:
💡 `processCommit` was widened from `private` to package-private to enable
the new test to call it directly from the same package. This is a clean
approach, but the Javadoc is missing `@param eventContext`. Please add it:
```suggestion
/**
* Strategy to commit the offset after message being processed
successfully.
*
* @param exchange the exchange
* @param eventContext the event context used for checkpointing
*/
void processCommit(final Exchange exchange, final EventContext
eventContext) {
```
--
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]
