Jackie-Jiang commented on issue #5829: URL: https://github.com/apache/incubator-pinot/issues/5829#issuecomment-812237787
For approach 1, my concern is more about changing the format in the json response, which might break the service parsing it. But approach 1 seems the better solution because if we only change our UI to handle it, other clients reading the result can also run into the same issue. For approach 1, there are 2 options: 1. Return all values as string (with double quotes) 2. Only convert large values to string IMO, option 1 has consistent format of the result (always have values double quoted), but will change the format of the current results; option 2 could return some values with double quotes and some without, but won't change the format for most of the results. Thoughts on this? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
