[ https://issues.apache.org/jira/browse/DRILL-4576?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15228750#comment-15228750 ]
ASF GitHub Bot commented on DRILL-4576: --------------------------------------- Github user laurentgo commented on the pull request: https://github.com/apache/drill/pull/466#issuecomment-206487918 Patch overall looks good to me (except maybe the abstract class vs interface stuff) > Add StoragePlugin API to register materialization into planner > -------------------------------------------------------------- > > Key: DRILL-4576 > URL: https://issues.apache.org/jira/browse/DRILL-4576 > Project: Apache Drill > Issue Type: Bug > Reporter: Laurent Goujon > Assignee: Jacques Nadeau > > There's no currently a good way to register materializations into Drill > planner. Calcite's MaterializationService.instance() would be the way to go, > but the registration happens in > {{org.apache.calcite.prepare.Prepare.PreparedResult#prepareSql()}}, which is > not called by Drill. -- This message was sent by Atlassian JIRA (v6.3.4#6332)