[jira] [Commented] (CALCITE-1871) Nesting LAST within PREV is not parsed correctly for MATCH_RECOGNIZE

2017-08-15 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1871?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16126861#comment-16126861 ] Dian Fu commented on CALCITE-1871: -- Hi [~ransom], just found that this patch disabled

[jira] [Created] (CALCITE-1943) Add back NavigationExpander and NavigationReplacer in SqlValidatorImpl

2017-08-15 Thread Dian Fu (JIRA)
Dian Fu created CALCITE-1943: Summary: Add back NavigationExpander and NavigationReplacer in SqlValidatorImpl Key: CALCITE-1943 URL: https://issues.apache.org/jira/browse/CALCITE-1943 Project: Calcite

[jira] [Commented] (CALCITE-1943) Add back NavigationExpander and NavigationReplacer in SqlValidatorImpl

2017-08-15 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1943?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16126885#comment-16126885 ] Dian Fu commented on CALCITE-1943: -- For {{NavigationExpander}}, currently I have not got a quite

[jira] [Commented] (CALCITE-1871) Nesting LAST within PREV is not parsed correctly for MATCH_RECOGNIZE

2017-08-15 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1871?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16126887#comment-16126887 ] Dian Fu commented on CALCITE-1871: -- Hi [~ransom], thanks for your reply. I have created an issue

[jira] [Commented] (CALCITE-1943) Add back NavigationExpander and NavigationReplacer in SqlValidatorImpl

2017-08-15 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1943?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16127023#comment-16127023 ] Dian Fu commented on CALCITE-1943: -- You're right. This is related to how to implement the above

[jira] [Commented] (CALCITE-1943) Add back NavigationExpander and NavigationReplacer in SqlValidatorImpl

2017-08-15 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1943?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16126989#comment-16126989 ] Dian Fu commented on CALCITE-1943: -- I have created a PR: https://github.com/apache/calcite/pull/516

[jira] [Commented] (CALCITE-1943) Add back NavigationExpander and NavigationReplacer in SqlValidatorImpl

2017-08-15 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1943?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16127038#comment-16127038 ] Dian Fu commented on CALCITE-1943: -- In one word, if we don't transform {{A.net_weight}} to

[jira] [Comment Edited] (CALCITE-1911) Support WITHIN clause in MATCH_RECOGNIZE

2017-08-11 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16123278#comment-16123278 ] Dian Fu edited comment on CALCITE-1911 at 8/11/17 12:38 PM: Have updated the

[jira] [Commented] (CALCITE-1911) Support WITHIN clause in MATCH_RECOGNIZE

2017-08-11 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16123278#comment-16123278 ] Dian Fu commented on CALCITE-1911: -- Have updated the PR according to review comments. Have also added

[jira] [Commented] (CALCITE-1935) Reference implementation for MATCH_RECOGNIZE

2017-08-11 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1935?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16123288#comment-16123288 ] Dian Fu commented on CALCITE-1935: -- [~julianhyde], sorry for late response. IMO, it's not easy to add

[jira] [Commented] (CALCITE-1911) Support WITHIN clause in MATCH_RECOGNIZE

2017-08-13 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16125149#comment-16125149 ] Dian Fu commented on CALCITE-1911: -- Make sense. Have updated the tests for INTERVAL. > Support WITHIN

[jira] [Updated] (CALCITE-1909) The output rowType should also include partition by key columns and order by key columns

2017-07-27 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1909?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dian Fu updated CALCITE-1909: - Description: According to the documentation, the output rowType should also include partition by key

[jira] [Updated] (CALCITE-1909) The output rowType should also include partition by key columns and order by key columns

2017-07-27 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1909?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dian Fu updated CALCITE-1909: - Description: According to the documentation, the output rowType should also include partition by key

[jira] [Updated] (CALCITE-1909) The output rowType should also include partition by and order by key columns

2017-07-27 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1909?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dian Fu updated CALCITE-1909: - Summary: The output rowType should also include partition by and order by key columns (was: The output

[jira] [Updated] (CALCITE-1909) The output rowType should also include partition by and order by key columns

2017-07-27 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1909?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dian Fu updated CALCITE-1909: - Description: According to the documentation, the output rowType should also include partition by key

[jira] [Updated] (CALCITE-1909) The output rowType should also include partition by key columns and order by key columns

2017-07-27 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1909?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dian Fu updated CALCITE-1909: - Description: According to the documentation, the output rowType should also include partition by key

[jira] [Updated] (CALCITE-1909) The output rowType should also include partition by key columns and order by key columns

2017-07-27 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1909?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dian Fu updated CALCITE-1909: - Description: According to the documentation, the output rowType should also include partition by key

[jira] [Updated] (CALCITE-1909) The output rowType should also include partition by key columns and order by key columns

2017-07-27 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1909?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dian Fu updated CALCITE-1909: - Description: According to the documentation, the output rowType should also include partition by key

[jira] [Created] (CALCITE-1909) The output rowType should also include partition by key columns and order by key columns

2017-07-27 Thread Dian Fu (JIRA)
Dian Fu created CALCITE-1909: Summary: The output rowType should also include partition by key columns and order by key columns Key: CALCITE-1909 URL: https://issues.apache.org/jira/browse/CALCITE-1909

[jira] [Updated] (CALCITE-1909) The output rowType should also include partition by key columns and order by key columns

2017-07-27 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1909?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dian Fu updated CALCITE-1909: - Description: According to the documentation, the output rowType should also include partition by key

[jira] [Created] (CALCITE-1871) Nesting LAST within PREV is not parsed correctly for MATCH_RECOGNIZE

2017-07-05 Thread Dian Fu (JIRA)
Dian Fu created CALCITE-1871: Summary: Nesting LAST within PREV is not parsed correctly for MATCH_RECOGNIZE Key: CALCITE-1871 URL: https://issues.apache.org/jira/browse/CALCITE-1871 Project: Calcite

[jira] [Created] (CALCITE-1872) PREV is parsed as LAST when the name of the pattern referenced is not the same with the pattern being defined

2017-07-05 Thread Dian Fu (JIRA)
Dian Fu created CALCITE-1872: Summary: PREV is parsed as LAST when the name of the pattern referenced is not the same with the pattern being defined Key: CALCITE-1872 URL:

[jira] [Created] (CALCITE-1911) Support within clause in MATCH_RECOGNIZE

2017-07-30 Thread Dian Fu (JIRA)
Dian Fu created CALCITE-1911: Summary: Support within clause in MATCH_RECOGNIZE Key: CALCITE-1911 URL: https://issues.apache.org/jira/browse/CALCITE-1911 Project: Calcite Issue Type: Bug

[jira] [Commented] (CALCITE-1911) Support within clause in MATCH_RECOGNIZE

2017-07-31 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16108289#comment-16108289 ] Dian Fu commented on CALCITE-1911: -- Right, I think the {{rowtime}} column can be seen as the timestamp

[jira] [Comment Edited] (CALCITE-1911) Support within clause in MATCH_RECOGNIZE

2017-07-31 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16108257#comment-16108257 ] Dian Fu edited comment on CALCITE-1911 at 8/1/17 1:36 AM: -- Correct, this is not

[jira] [Commented] (CALCITE-1911) Support within clause in MATCH_RECOGNIZE

2017-07-31 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16108257#comment-16108257 ] Dian Fu commented on CALCITE-1911: -- Yes, this is not in Oracle 12 and also not in the SQL standard, but

[jira] [Commented] (CALCITE-1911) Support WITHIN clause in MATCH_RECOGNIZE

2017-08-08 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16118240#comment-16118240 ] Dian Fu commented on CALCITE-1911: -- [~julianhyde], [~ransom] Thanks a lot for your review. {quote} I am

[jira] [Commented] (CALCITE-1911) Support within clause in MATCH_RECOGNIZE

2017-08-07 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16116608#comment-16116608 ] Dian Fu commented on CALCITE-1911: -- [~julianhyde] [~ransom], Have create a PR. Could you help to

[jira] [Commented] (CALCITE-1911) Support WITHIN clause in MATCH_RECOGNIZE

2017-08-22 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16137738#comment-16137738 ] Dian Fu commented on CALCITE-1911: -- Hi [~julianhyde], I have addressed your comments in the latest PR

[jira] [Resolved] (CALCITE-1911) Support WITHIN clause in MATCH_RECOGNIZE

2017-08-24 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dian Fu resolved CALCITE-1911. -- Resolution: Fixed Fix Version/s: 1.14.0 > Support WITHIN clause in MATCH_RECOGNIZE >

[jira] [Commented] (CALCITE-1943) Add back NavigationExpander and NavigationReplacer in SqlValidatorImpl

2017-08-24 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1943?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16141088#comment-16141088 ] Dian Fu commented on CALCITE-1943: -- [~julianhyde], I have updated the PR, documented the rationale at

[jira] [Commented] (CALCITE-1986) Add RelBuilder.matchRecognize

2017-09-16 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1986?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16168920#comment-16168920 ] Dian Fu commented on CALCITE-1986: -- [~julianhyde] I have left a few minor comments on your branch. The

[jira] [Created] (CALCITE-1986) Add RelBuilder.matchRecognize

2017-09-13 Thread Dian Fu (JIRA)
Dian Fu created CALCITE-1986: Summary: Add RelBuilder.matchRecognize Key: CALCITE-1986 URL: https://issues.apache.org/jira/browse/CALCITE-1986 Project: Calcite Issue Type: Bug

[jira] [Commented] (CALCITE-1986) Add RelBuilder.matchRecognize

2017-09-13 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1986?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16165739#comment-16165739 ] Dian Fu commented on CALCITE-1986: -- [~julianhyde] Could you help to review when it's convenient for

[jira] [Commented] (CALCITE-1986) Add RelBuilder.matchRecognize

2017-09-14 Thread Dian Fu (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-1986?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16165840#comment-16165840 ] Dian Fu commented on CALCITE-1986: -- {quote} when do we need to create a matchrecoginize relNode used