amrishlal commented on pull request #6403: URL: https://github.com/apache/incubator-pinot/pull/6403#issuecomment-768498727
> Per the performance concern, can we always parse the String as double? The main reason why I used BigDecimal was that long to double to long conversion does not give back the original value since both are 64bit. I will look into this a bit more, but it is probably safer to lean in the direction of query correctness as compared to performance. ---------------------------------------------------------------- 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 --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
