eolivelli opened a new pull request #10248:
URL: https://github.com/apache/pulsar/pull/10248


   ### Motivation
   I saw this problem while developing integration tests for GenericObject + 
KeyValue with SEPARATED KeyValueEncoding.
   
   I have extracted this patch in order to make it clearer, it was part of 
#10211 in which I have dropped the tests with SEPARATED KeyValueEncoding due to 
this problem.
   
   ### Modifications
   
   Before executing MessageImpl#getValue we are now ensuring that the Schema is 
really loaded when we call getSchemaInfo.
   
   Initially I added an eager loading of the Schema in getSchemaInfo, but this 
resulted in lots of deadlocks because we are calling getSchemaInfo in many 
places. 
   The code also expects that AutoConsumeSchema returns getSchemaInfo == null 
in several places and we have to keep this behaviour.
   
   ### Verifying this change
   
   I am not adding test cases, I will add test cases in a follow up patch, once 
#10211 lands to master
   


-- 
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.

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


Reply via email to