Chickenzilla opened a new pull request #50: URL: https://github.com/apache/pulsar-dotpulsar/pull/50
In a confusingly ironic twist, using the new nullable language features in a public-facing area of this library has caused a bug where if a consumer checks a `MessageId` against `null` it will throw a NullReferenceException, which is pretty much the worst thing it could do in that scenario. Since we target .net standard 2.0 and no particular language version as a library, we should be tolerant of such a check, even if it is not possible for that value to be null. ---------------------------------------------------------------- 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
