Re: [DISCUSS] Default disable the RexNode normalization(or operands reorder)

2020-07-13 Thread Danny Chan
Hi, all, I’m planning to default disable the RexNode normalization in CALCITE-4073, if you have any objections, please let me know in 24 hours, thanks so much ~ Looking forward to your feedback ~ Best, Danny Chan 在 2020年7月8日 +0800 PM4:38,Danny Chan ,写道: > In CALCITE-2450, we proposed a change t

Re: [DISCUSS] Default disable the RexNode normalization(or operands reorder)

2020-07-13 Thread Vladimir Sitnikov
>Hi, all, I’m planning to default disable the RexNode normalization in CALCITE-4073, if you have any objections, please let me know in 24 hours, thanks so much ~ I assume it would still normalize RexNodes when building plan digest. Is it the case? Vladimir

Re: [DISCUSS] Default disable the RexNode normalization(or operands reorder)

2020-07-13 Thread Danny Chan
Yes, it is. We can keep it as a builtin promotion. Best, Danny Chan 在 2020年7月13日 +0800 PM3:48,Vladimir Sitnikov ,写道: > > Hi, all, I’m planning to default disable the RexNode normalization in > CALCITE-4073, if you have any objections, please let me know in 24 hours, > thanks so much ~ > > I assume

Re: Calcite PR CI offten failed due to elasticsearch:test

2020-07-13 Thread Rui Wang
Here Andrei, Here is an example of a failed PR check that was failed due to elasticsearch:test: https://ci.appveyor.com/project/ApacheSoftwareFoundation/calcite/builds/33662741 See that failed Jdk11 check log. -Rui On Sun, Jul 12, 2020 at 4:31 PM Andrei Sereda wrote: > Hello, > > Can somebod