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 2019/01/25 21:06:38 UTC
[GitHub] AlexanderSaydakov commented on issue #6893: Add several missing
inspectRuntimeShape() calls
AlexanderSaydakov commented on issue #6893: Add several missing inspectRuntimeShape() calls
URL: https://github.com/apache/incubator-druid/pull/6893#issuecomment-457728344
What is the cost of making a mistake both ways?
Do I understand correctly the above comment by drcrallen that missing parameters that do affect the runtime shape would miss a potential performance improvement? What about adding unnecessary ones? Any possibility to affect correctness of the results?
----------------------------------------------------------------
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