[jira] [Commented] (CALCITE-4060) TypeCoercionImpl#inOperationCoercion doesn't consider NOT_IN.

2020-06-15 Thread Jiatao Tao (Jira)


[ 
https://issues.apache.org/jira/browse/CALCITE-4060?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17136247#comment-17136247
 ] 

Jiatao Tao commented on CALCITE-4060:
-

Hi [~julianhyde] , you can take a look at 
"org.apache.calcite.test.TypeCoercionConverterTest#testInOperation", and debug 
in 
"org.apache.calcite.sql.validate.implicit.TypeCoercionImpl#inOperationCoercion".

> TypeCoercionImpl#inOperationCoercion doesn't consider NOT_IN.
> -
>
> Key: CALCITE-4060
> URL: https://issues.apache.org/jira/browse/CALCITE-4060
> Project: Calcite
>  Issue Type: Bug
>Reporter: Jiatao Tao
>Priority: Critical
> Attachments: image-2020-06-12-10-22-49-218.png, 
> image-2020-06-12-10-43-52-584.png
>
>  Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> Current TypeCoercionImpl#inOperationCoercion only considers IN.
> !image-2020-06-12-10-43-52-584.png!



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (CALCITE-4060) TypeCoercionImpl#inOperationCoercion doesn't consider NOT_IN.

2020-06-12 Thread Julian Hyde (Jira)


[ 
https://issues.apache.org/jira/browse/CALCITE-4060?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17134414#comment-17134414
 ] 

Julian Hyde commented on CALCITE-4060:
--

What's the test case?

> TypeCoercionImpl#inOperationCoercion doesn't consider NOT_IN.
> -
>
> Key: CALCITE-4060
> URL: https://issues.apache.org/jira/browse/CALCITE-4060
> Project: Calcite
>  Issue Type: Bug
>Reporter: Jiatao Tao
>Priority: Critical
> Attachments: image-2020-06-12-10-22-49-218.png, 
> image-2020-06-12-10-43-52-584.png
>
>  Time Spent: 20m
>  Remaining Estimate: 0h
>
> Current TypeCoercionImpl#inOperationCoercion only considers IN.
> !image-2020-06-12-10-43-52-584.png!



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (CALCITE-4060) TypeCoercionImpl#inOperationCoercion doesn't consider NOT_IN.

2020-06-11 Thread Jiatao Tao (Jira)


[ 
https://issues.apache.org/jira/browse/CALCITE-4060?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17133853#comment-17133853
 ] 

Jiatao Tao commented on CALCITE-4060:
-

[~amaliujia] I add a ut: TypeCoercionConverterTest#testNotInOperation, imitate 
"TypeCoercionConverterTest#testInOperation"

> TypeCoercionImpl#inOperationCoercion doesn't consider NOT_IN.
> -
>
> Key: CALCITE-4060
> URL: https://issues.apache.org/jira/browse/CALCITE-4060
> Project: Calcite
>  Issue Type: Bug
>Reporter: Jiatao Tao
>Priority: Critical
> Attachments: image-2020-06-12-10-22-49-218.png
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (CALCITE-4060) TypeCoercionImpl#inOperationCoercion doesn't consider NOT_IN.

2020-06-11 Thread Jiatao Tao (Jira)


[ 
https://issues.apache.org/jira/browse/CALCITE-4060?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17133846#comment-17133846
 ] 

Jiatao Tao commented on CALCITE-4060:
-

Hi [~julianhyde], I add a ut, and debug we can find the op is NOT IN:

org.apache.calcite.test.TypeCoercionConverterTest#testNotInOperation

!image-2020-06-12-10-22-49-218.png!

> TypeCoercionImpl#inOperationCoercion doesn't consider NOT_IN.
> -
>
> Key: CALCITE-4060
> URL: https://issues.apache.org/jira/browse/CALCITE-4060
> Project: Calcite
>  Issue Type: Bug
>Reporter: Jiatao Tao
>Priority: Critical
> Attachments: image-2020-06-12-10-22-49-218.png
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (CALCITE-4060) TypeCoercionImpl#inOperationCoercion doesn't consider NOT_IN.

2020-06-11 Thread Julian Hyde (Jira)


[ 
https://issues.apache.org/jira/browse/CALCITE-4060?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17133771#comment-17133771
 ] 

Julian Hyde commented on CALCITE-4060:
--

I don't think NOT_IN should exist inside RexNodes. See CALCITE-3242.


> TypeCoercionImpl#inOperationCoercion doesn't consider NOT_IN.
> -
>
> Key: CALCITE-4060
> URL: https://issues.apache.org/jira/browse/CALCITE-4060
> Project: Calcite
>  Issue Type: Bug
>Reporter: Jiatao Tao
>Priority: Critical
>  Time Spent: 10m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (CALCITE-4060) TypeCoercionImpl#inOperationCoercion doesn't consider NOT_IN.

2020-06-11 Thread Rui Wang (Jira)


[ 
https://issues.apache.org/jira/browse/CALCITE-4060?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17133763#comment-17133763
 ] 

Rui Wang commented on CALCITE-4060:
---

[~Aron.tao] Could you use some examples in this Jira to demonstrate your 
intention? E.g. use an query to demonstrate what is wrong?

> TypeCoercionImpl#inOperationCoercion doesn't consider NOT_IN.
> -
>
> Key: CALCITE-4060
> URL: https://issues.apache.org/jira/browse/CALCITE-4060
> Project: Calcite
>  Issue Type: Bug
>Reporter: Jiatao Tao
>Priority: Critical
>  Time Spent: 10m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (CALCITE-4060) TypeCoercionImpl#inOperationCoercion doesn't consider NOT_IN.

2020-06-11 Thread Jiatao Tao (Jira)


[ 
https://issues.apache.org/jira/browse/CALCITE-4060?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17133280#comment-17133280
 ] 

Jiatao Tao commented on CALCITE-4060:
-

MR: [https://github.com/apache/calcite/pull/2020]

> TypeCoercionImpl#inOperationCoercion doesn't consider NOT_IN.
> -
>
> Key: CALCITE-4060
> URL: https://issues.apache.org/jira/browse/CALCITE-4060
> Project: Calcite
>  Issue Type: Bug
>Reporter: Jiatao Tao
>Priority: Critical
>  Time Spent: 10m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)