eolivelli edited a comment on pull request #9900: URL: https://github.com/apache/pulsar/pull/9900#issuecomment-799515715
@315157973 the mutation function is execution inside this kind of optimistic "lock" here we get the version and we store it in a stack local variable https://github.com/apache/pulsar/blob/5f77745e05ec919a89060bee2ba805ec847ac7e0/pulsar-metadata/src/main/java/org/apache/pulsar/metadata/cache/impl/MetadataCacheImpl.java#L163 then we write to ZK https://github.com/apache/pulsar/blob/5f77745e05ec919a89060bee2ba805ec847ac7e0/pulsar-metadata/src/main/java/org/apache/pulsar/metadata/cache/impl/MetadataCacheImpl.java#L175 so there is no need to add a new version variable to Policies IMHO ---------------------------------------------------------------- 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: [email protected]
