You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/03/04 08:00:19 UTC

[GitHub] [pulsar] lhotari commented on issue #14436: [Proto] java.lang.IllegalStateException: Some required fields are missing

lhotari commented on issue #14436:
URL: https://github.com/apache/pulsar/issues/14436#issuecomment-1058925456


   I made an attempt to fix this issue in #14556.
   
   The `IllegalStateException: Some required fields are missing` could happen if either `publishTime`, `producerName` or `sequenceId` is unset.
   
   https://github.com/apache/pulsar/blob/db25438199b0dfc39719a7743b861917b5fbd833/pulsar-common/src/main/proto/PulsarApi.proto#L105-L107
   
   One interesting detail is this code:
   
   https://github.com/apache/pulsar/blob/5bcdb5ef40a24fefad042efac484f2be4b92c527/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ProducerImpl.java#L565-L577
   
   I guess that's fine (similar logic has been in place since the initial commit). The problem is more likely a thread safety issue or shared thread local instance issue. That's why #14556 could be effective.


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

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