You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by GitBox <gi...@apache.org> on 2019/04/07 19:14:12 UTC

[GitHub] [drill] amansinha100 commented on issue #1728: DRILL-7089: Implement caching for TableMetadataProvider at query level and adapt statistics to use Drill metastore API

amansinha100 commented on issue #1728: DRILL-7089: Implement caching for TableMetadataProvider at query level and adapt statistics to use Drill metastore API
URL: https://github.com/apache/drill/pull/1728#issuecomment-480620573
 
 
   @vvysotskyi one other thought is that since this PR touches several of the Statistics related classes,  considering the Drill 1.16.0 release is imminent and the core statistics testing is ongoing, it would be disruptive to introduce these changes at this stage.  Even though functional regression might pass, we have had performance issues even with statistics disabled, so I feel hesitant for 1.16.0.  Let me know what you think.  

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services