jackjlli commented on issue #4127: Declare byte array size when initializing ByteArrayOutputStream URL: https://github.com/apache/incubator-pinot/pull/4127#issuecomment-484272915 @sunithabeeram * Right now we don't have any metric of tracking the response size yet. It'd be good to add one. * We can do the same for serializing metadata, dictionary and data schema. But there may be several `for` loops in those methods, which need to create multiple hashmaps to track the byte[] beforehand. * We've already had unit tests on DataTable, so we're good here.
---------------------------------------------------------------- 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] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
