[GitHub] [camel] orpiske commented on pull request #7503: CAMEL-18019: ensure some fields in the Kafka record fetcher are safe for concurrent read

[GitBox <gi...@apache.org>]

orpiske commented on PR #7503:
URL: https://github.com/apache/camel/pull/7503#issuecomment-1111125210

   > What about `clientId` and `lastError`? They seem to be shared also
   
   Hm, `clientId` seems to be written only within the scope of the same thread (in 
   
   > I see also a potential bug [here](https://github.com/apache/camel/blob/b5361d38f0d56590d151a654b11920c463beaa3b/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaFetchRecords.java#L104). Indeed the `pollExceptionStrategy` is set in the constructor using the value of the field `consumer` that is always `null` at this stage
   
   
   
   > I see also a potential bug [here](https://github.com/apache/camel/blob/b5361d38f0d56590d151a654b11920c463beaa3b/components/camel-kafka/src/main/java/org/apache/camel/component/kafka/KafkaFetchRecords.java#L104). Indeed the `pollExceptionStrategy` is set in the constructor using the value of the field `consumer` that is always `null` at this stage
   
   Please open a separate ticket for this one.


-- 
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-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org