[
https://issues.apache.org/jira/browse/CALCITE-2662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16708236#comment-16708236
]
Enrico Olivelli commented on CALCITE-2662:
--
Thank you Julian and Vladmin for review
>
[
https://issues.apache.org/jira/browse/CALCITE-2662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16706568#comment-16706568
]
Julian Hyde commented on CALCITE-2662:
--
Reviewing and testing now.
> Planner: allow parsing
[
https://issues.apache.org/jira/browse/CALCITE-2662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16705766#comment-16705766
]
Enrico Olivelli commented on CALCITE-2662:
--
[~julianhyde] patch is ready from my side. (sorry
[
https://issues.apache.org/jira/browse/CALCITE-2662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16704932#comment-16704932
]
Julian Hyde commented on CALCITE-2662:
--
It's OK if you provide "?" if someone asks for the
[
https://issues.apache.org/jira/browse/CALCITE-2662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16703710#comment-16703710
]
Enrico Olivelli commented on CALCITE-2662:
--
Just because I did not see any test case about the
[
https://issues.apache.org/jira/browse/CALCITE-2662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16703687#comment-16703687
]
Julian Hyde commented on CALCITE-2662:
--
Why do you think it is not useful for the user to see the
[
https://issues.apache.org/jira/browse/CALCITE-2662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16703682#comment-16703682
]
Enrico Olivelli commented on CALCITE-2662:
--
I will add tests.
As 'originalinput' query seems
[
https://issues.apache.org/jira/browse/CALCITE-2662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16703592#comment-16703592
]
Julian Hyde commented on CALCITE-2662:
--
The fact that you are able to use a StringReader is a good
[
https://issues.apache.org/jira/browse/CALCITE-2662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16703362#comment-16703362
]
Enrico Olivelli commented on CALCITE-2662:
--
[~julianhyde] I have added a Proof-of-concept
[
https://issues.apache.org/jira/browse/CALCITE-2662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16699484#comment-16699484
]
Enrico Olivelli commented on CALCITE-2662:
--
Great. I will add the test soon
This is the
[
https://issues.apache.org/jira/browse/CALCITE-2662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16699459#comment-16699459
]
Julian Hyde commented on CALCITE-2662:
--
Can you attach a link to the PR?
If the PR is
[
https://issues.apache.org/jira/browse/CALCITE-2662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16697810#comment-16697810
]
Enrico Olivelli commented on CALCITE-2662:
--
Sorry for late reply.
I have updated the patch
[
https://issues.apache.org/jira/browse/CALCITE-2662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16680695#comment-16680695
]
Julian Hyde commented on CALCITE-2662:
--
I'm skeptical that this will give a significant
[
https://issues.apache.org/jira/browse/CALCITE-2662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16679378#comment-16679378
]
Vladimir Sitnikov commented on CALCITE-2662:
Would you probably add a test case to
[
https://issues.apache.org/jira/browse/CALCITE-2662?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16679377#comment-16679377
]
Vladimir Sitnikov commented on CALCITE-2662:
[~eolivelli], have you seen
15 matches
Mail list logo