[jira] [Commented] (CALCITE-2430) AssertionError: RelDataTypeImpl.getFieldList when SQL Advisor inspects non-struct field

2018-07-30 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2430?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16562716#comment-16562716 ] Julian Hyde commented on CALCITE-2430: -- By the way, I totally agree about advisor needing to be

[jira] [Commented] (CALCITE-2430) AssertionError: RelDataTypeImpl.getFieldList when SQL Advisor inspects non-struct field

2018-07-30 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2430?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16562709#comment-16562709 ] Julian Hyde commented on CALCITE-2430: -- Yes, I think it's worth doing. Could you do it as a

[jira] [Commented] (CALCITE-2430) AssertionError: RelDataTypeImpl.getFieldList when SQL Advisor inspects non-struct field

2018-07-30 Thread Vladimir Sitnikov (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2430?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16562519#comment-16562519 ] Vladimir Sitnikov commented on CALCITE-2430: {quote}Yes, I know. But is there any point in

[jira] [Commented] (CALCITE-2430) AssertionError: RelDataTypeImpl.getFieldList when SQL Advisor inspects non-struct field

2018-07-30 Thread Vladimir Sitnikov (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2430?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16562495#comment-16562495 ] Vladimir Sitnikov commented on CALCITE-2430: I think this is relevant:

[jira] [Commented] (CALCITE-2430) AssertionError: RelDataTypeImpl.getFieldList when SQL Advisor inspects non-struct field

2018-07-30 Thread Vladimir Sitnikov (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2430?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16562486#comment-16562486 ] Vladimir Sitnikov commented on CALCITE-2430: Changing SqlValidatorImpl as follows:

[jira] [Commented] (CALCITE-2430) AssertionError: RelDataTypeImpl.getFieldList when SQL Advisor inspects non-struct field

2018-07-30 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2430?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16562484#comment-16562484 ] Julian Hyde commented on CALCITE-2430: -- Yes, I know. But is there any point in having a namespace

[jira] [Commented] (CALCITE-2430) AssertionError: RelDataTypeImpl.getFieldList when SQL Advisor inspects non-struct field

2018-07-30 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2430?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16562440#comment-16562440 ] Julian Hyde commented on CALCITE-2430: -- bq. It looks like it makes sense to add {{isStruct}} check