You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2018/10/03 02:59:45 UTC
[GitHub] jihoonson commented on issue #6389: Replace MapBasedRow with
ArrayBasedRow for groupBy
jihoonson commented on issue #6389: Replace MapBasedRow with ArrayBasedRow for groupBy
URL: https://github.com/apache/incubator-druid/issues/6389#issuecomment-426496524
Hi @PannerselvamVelmyl. Basically yes, it's correct. But I think there're something more we can do. For example, with ArrayBasedRow, the result column order is always fixed, so that we can avoid serializing/deserializing the column names when sending intermediate data from historicals to the broker. This would reduce the intermediate data size.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org