richardstartin opened a new pull request, #8499: URL: https://github.com/apache/pinot/pull/8499
Calling `Enum.values()` allocates an array every time it is called, and this method gets called a lot during `DataTable` deserialization, which shows up in allocation profiles. Also deserializes table directly from `ByteBuffer` to avoid allocating lots of `byte[]` during deserialization. This isn't done when the `DataTable` retains a copy of the `ByteBuffer` because of ambiguity of ownership, though it looks like it could be done safely. -- 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
