aymkhalil commented on issue #20092:
URL: https://github.com/apache/pulsar/issues/20092#issuecomment-1516917097
> I think we don't need to think about JsonNode.get("xxxx") it's not the API
of Pulsar
Ideally yes, but my understanding is that the public API is limited and
hence nativeObject was exposed (as far as end user is concerned this is part of
the API no?).
With that in mind, I think it is safer if the native object and the public
API behavior match (or I can't think of a good reason why they wouldn't).
> if is the FloatingPointNumber we return double type directly if the field
schema is decimal, we should return decimal
This actually demonstrates the confusion because the generic pulsar schema
API doesn't have a DECIMAL type. As a user, I think of nativeObject as my
source of truth. Let me know if this is the wrong expectation.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
