[
https://issues.apache.org/jira/browse/CALCITE-4586?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jiatao Tao updated CALCITE-4586:
--------------------------------
Description:
Hi, piglet.PigRelBuilder#create, we set Hook.REL_BUILDER_SIMPLIFY to false:
This may be not a suitable place to set "Hook.REL_BUILDER_SIMPLIFY", we should
use this in user code, not in the frame code, cuz once we done this in frame
code, we can not change this config anymore.
This change may be perceived if you are using piglet.PigRelBuilder,
it is expected that the change will not be large and should be positive, cuz
the UTs are almost unchanged and all passed.
You can set "config.simplify()" to false to keep the behavior the same
was:
Hi, piglet.PigRelBuilder#create, we set Hook.REL_BUILDER_SIMPLIFY to false:
This may be not a suitable place to set "Hook.REL_BUILDER_SIMPLIFY", we should
use this in user code, not in the frame code, cuz once we done this in frame
code, we can not change this config anymore.
This change may be perceived if you are using piglet.PigRelBuilder,
it is expected that the change will not be large and should be positive, cuz
the UTs are almost unchanged and all passed.
You can set
> In piglet, allow creating a PigRelBuilder with custom "config.simplify()"
> -------------------------------------------------------------------------
>
> Key: CALCITE-4586
> URL: https://issues.apache.org/jira/browse/CALCITE-4586
> Project: Calcite
> Issue Type: Improvement
> Components: core
> Reporter: Jiatao Tao
> Assignee: Jiatao Tao
> Priority: Critical
> Fix For: 1.27.0
>
>
> Hi, piglet.PigRelBuilder#create, we set Hook.REL_BUILDER_SIMPLIFY to false:
> This may be not a suitable place to set "Hook.REL_BUILDER_SIMPLIFY", we
> should use this in user code, not in the frame code, cuz once we done this in
> frame code, we can not change this config anymore.
>
> This change may be perceived if you are using piglet.PigRelBuilder,
> it is expected that the change will not be large and should be positive, cuz
> the UTs are almost unchanged and all passed.
>
> You can set "config.simplify()" to false to keep the behavior the same
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
