[ 
https://issues.apache.org/jira/browse/DRILL-7523?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17567900#comment-17567900
 ] 

ASF GitHub Bot commented on DRILL-7523:
---------------------------------------

vvysotskyi opened a new pull request, #2602:
URL: https://github.com/apache/drill/pull/2602

   # [DRILL-7523](https://issues.apache.org/jira/browse/DRILL-7523): Update 
Calcite to 1.31.0
   
   ## Description
   Changes required for updating Calcite to 1.31.0. For now, the master version 
built by JitPack is used. After Calcite 1.31.0 is released, this PR will be 
updated to use that version.
   
   No Drill-specific commits in Calcite are required after this change, so the 
https://github.com/vvysotskyi/drill-calcite repository will be archived after 
merging this PR.
   
   ## Documentation
   NA
   
   ## Testing
   All unit tests pass.
   




> Update Calcite to 1.31.0
> ------------------------
>
>                 Key: DRILL-7523
>                 URL: https://issues.apache.org/jira/browse/DRILL-7523
>             Project: Apache Drill
>          Issue Type: Task
>    Affects Versions: 1.17.0
>            Reporter: Vova Vysotskyi
>            Assignee: Vova Vysotskyi
>            Priority: Major
>             Fix For: Future
>
>
> Upgrade to Calcite 1.31 version.
> Also, there are some fixes that may help to avoid specific commit for 
> CALCITE-3121.
> As discussed in CALCITE-2223, there were made additional changes for fixing 
> this issue, so we should check whether we can remove the workaround made in 
> DRILL-6212.
> Incorporate with changes made in 
> https://issues.apache.org/jira/browse/CALCITE-3774 - remove overriding 
> {{shouldMergeProject}} and set {{withBloat(int bloat)}} in custom rel builder 
> and investigate whether it would help to remove {{Hook.REL_BUILDER_SIMPLIFY}} 
> hook.
> Additionally, please check whether specific commit with ViewExpander may be 
> removed due to changes made in CALCITE-2441.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to