Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-08 Thread via GitHub
orpiske commented on PR #11920: URL: https://github.com/apache/camel/pull/11920#issuecomment-1802377348 This needs to be backported to 3.x -- 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 spe

Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-08 Thread via GitHub
orpiske merged PR #11920: URL: https://github.com/apache/camel/pull/11920 -- 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: commits-unsubscr...@camel.apac

Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-08 Thread via GitHub
github-actions[bot] commented on PR #11920: URL: https://github.com/apache/camel/pull/11920#issuecomment-1802371276 :no_entry_sign: There are (likely) no changes in core core to be tested in this PR -- This is an automated message from the Apache Git Service. To respond to the message, pl

Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-08 Thread via GitHub
github-actions[bot] commented on PR #11920: URL: https://github.com/apache/camel/pull/11920#issuecomment-1802371246 ### Components test results: | Total | Tested | Failed :x: | Passed :white_check_mark: | | --- | --- | --- | --- | | 2 | 2 | 0 | 2 | -- This is an automated me

Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-08 Thread via GitHub
github-actions[bot] commented on PR #11920: URL: https://github.com/apache/camel/pull/11920#issuecomment-1802115592 :no_entry_sign: There are (likely) no changes in core core to be tested in this PR -- This is an automated message from the Apache Git Service. To respond to the message, pl

Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-08 Thread via GitHub
github-actions[bot] commented on PR #11920: URL: https://github.com/apache/camel/pull/11920#issuecomment-1802115561 ### Components test results: | Total | Tested | Failed :x: | Passed :white_check_mark: | | --- | --- | --- | --- | | 2 | 2 | 1 | 1 | -- This is an automated me

Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-08 Thread via GitHub
orpiske commented on PR #11920: URL: https://github.com/apache/camel/pull/11920#issuecomment-1802016189 Thank you @CodeSmell! I am now running manual tests on this code and will merge / report back soon. -- This is an automated message from the Apache Git Service. To respond to the messag

Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-08 Thread via GitHub
CodeSmell commented on PR #11920: URL: https://github.com/apache/camel/pull/11920#issuecomment-1802009407 > This is looking OK. @orpiske I updated the logging here so that it each word is capitalized per other comment -- This is an automated message from the Apache Git Service

Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-08 Thread via GitHub
orpiske commented on PR #11920: URL: https://github.com/apache/camel/pull/11920#issuecomment-1801458023 This is looking OK. -- 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.

Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-07 Thread via GitHub
github-actions[bot] commented on PR #11920: URL: https://github.com/apache/camel/pull/11920#issuecomment-1799335262 ### Components test results: | Total | Tested | Failed :x: | Passed :white_check_mark: | | --- | --- | --- | --- | | 2 | 2 | 1 | 1 | -- This is an automated me

Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-07 Thread via GitHub
github-actions[bot] commented on PR #11920: URL: https://github.com/apache/camel/pull/11920#issuecomment-1799335283 :no_entry_sign: There are (likely) no changes in core core to be tested in this PR -- This is an automated message from the Apache Git Service. To respond to the message, pl

Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-07 Thread via GitHub
github-actions[bot] commented on PR #11920: URL: https://github.com/apache/camel/pull/11920#issuecomment-1799303223 ### Components test results: | Total | Tested | Failed :x: | Passed :white_check_mark: | | --- | --- | --- | --- | | 2 | 2 | 1 | 1 | -- This is an automated me

Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-07 Thread via GitHub
github-actions[bot] commented on PR #11920: URL: https://github.com/apache/camel/pull/11920#issuecomment-1799303263 :no_entry_sign: There are (likely) no changes in core core to be tested in this PR -- This is an automated message from the Apache Git Service. To respond to the message, pl

Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-07 Thread via GitHub
orpiske commented on code in PR #11920: URL: https://github.com/apache/camel/pull/11920#discussion_r1385250145 ## components/camel-kafka/src/main/java/org/apache/camel/component/kafka/consumer/support/KafkaRecordProcessor.java: ## @@ -112,30 +115,48 @@ public ProcessingResult pr

Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-07 Thread via GitHub
CodeSmell commented on code in PR #11920: URL: https://github.com/apache/camel/pull/11920#discussion_r1384965058 ## components/camel-kafka/src/main/java/org/apache/camel/component/kafka/consumer/support/KafkaRecordProcessor.java: ## @@ -112,30 +115,48 @@ public ProcessingResult

Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-07 Thread via GitHub
orpiske commented on code in PR #11920: URL: https://github.com/apache/camel/pull/11920#discussion_r1384959149 ## components/camel-kafka/src/main/java/org/apache/camel/component/kafka/consumer/support/KafkaRecordProcessor.java: ## @@ -112,30 +115,48 @@ public ProcessingResult pr

Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-07 Thread via GitHub
CodeSmell commented on code in PR #11920: URL: https://github.com/apache/camel/pull/11920#discussion_r1384954679 ## components/camel-kafka/src/main/java/org/apache/camel/component/kafka/consumer/support/KafkaRecordProcessor.java: ## @@ -112,30 +115,48 @@ public ProcessingResult

Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-07 Thread via GitHub
orpiske commented on code in PR #11920: URL: https://github.com/apache/camel/pull/11920#discussion_r1384928509 ## components/camel-kafka/src/main/java/org/apache/camel/component/kafka/consumer/support/KafkaRecordProcessor.java: ## @@ -112,30 +115,48 @@ public ProcessingResult pr

Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-07 Thread via GitHub
orpiske commented on code in PR #11920: URL: https://github.com/apache/camel/pull/11920#discussion_r1384927769 ## components/camel-kafka/src/main/java/org/apache/camel/component/kafka/consumer/support/KafkaRecordProcessor.java: ## @@ -112,30 +115,48 @@ public ProcessingResult pr

Re: [PR] CAMEL-20044 add extra logging for BreakOnFirstError [camel]

2023-11-07 Thread via GitHub
CodeSmell commented on code in PR #11920: URL: https://github.com/apache/camel/pull/11920#discussion_r1384905892 ## components/camel-kafka/src/main/java/org/apache/camel/component/kafka/consumer/support/KafkaRecordProcessor.java: ## @@ -112,30 +115,48 @@ public ProcessingResult