Jackie-Jiang opened a new pull request, #11716: URL: https://github.com/apache/pinot/pull/11716
It is best practice to store `BigDecimal` as string in JSON to prevent losing precision. Currently we directly serialize `BigDecimal` into number (without quote) in the query response, but `JsonAsyncHttpPinotClientTransport` will parse it into double and lose precision. Instead of fixing the client, fixing the `BigDecimal` JSON storage is the recommended solution since JSON number is not designed to preserve precision, and a lot of clients might run into the same problem. -- 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-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
