[jira] [Commented] (PHOENIX-3242) Support CREATE OR REPLACE FUNCTION in Phoenix-Calcite Integration

2016-10-10 Thread Rajeshbabu Chintaguntla (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15563612#comment-15563612 ] Rajeshbabu Chintaguntla commented on PHOENIX-3242: -- bq. does our FunctionResolver

[jira] [Commented] (PHOENIX-3242) Support CREATE OR REPLACE FUNCTION in Phoenix-Calcite Integration

2016-10-10 Thread Maryann Xue (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15563568#comment-15563568 ] Maryann Xue commented on PHOENIX-3242: -- You also need to check how well view extension columns are

[jira] [Commented] (PHOENIX-3242) Support CREATE OR REPLACE FUNCTION in Phoenix-Calcite Integration

2016-10-10 Thread Maryann Xue (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15563560#comment-15563560 ] Maryann Xue commented on PHOENIX-3242: -- Thank you very much, [~rajeshbabu]! One question though,

[jira] [Commented] (PHOENIX-3242) Support CREATE OR REPLACE FUNCTION in Phoenix-Calcite Integration

2016-10-10 Thread Rajeshbabu Chintaguntla (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15563444#comment-15563444 ] Rajeshbabu Chintaguntla commented on PHOENIX-3242: -- [~maryannxue] Thanks for review. I

[jira] [Commented] (PHOENIX-3242) Support CREATE OR REPLACE FUNCTION in Phoenix-Calcite Integration

2016-10-06 Thread Maryann Xue (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15553369#comment-15553369 ] Maryann Xue commented on PHOENIX-3242: -- Thank you very much, for your nice work, [~rajeshbabu]! I

[jira] [Commented] (PHOENIX-3242) Support CREATE OR REPLACE FUNCTION in Phoenix-Calcite Integration

2016-09-30 Thread Maryann Xue (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15536887#comment-15536887 ] Maryann Xue commented on PHOENIX-3242: -- [~rajeshbabu], turns out that the UPSERT issue and the DROP

[jira] [Commented] (PHOENIX-3242) Support CREATE OR REPLACE FUNCTION in Phoenix-Calcite Integration

2016-09-30 Thread Rajeshbabu Chintaguntla (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15535214#comment-15535214 ] Rajeshbabu Chintaguntla commented on PHOENIX-3242: -- [~maryannxue] Thanks for updated

[jira] [Commented] (PHOENIX-3242) Support CREATE OR REPLACE FUNCTION in Phoenix-Calcite Integration

2016-09-29 Thread Maryann Xue (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15534432#comment-15534432 ] Maryann Xue commented on PHOENIX-3242: -- Thank you, [~rajeshbabu], for the WIP patch! I have a quick

[jira] [Commented] (PHOENIX-3242) Support CREATE OR REPLACE FUNCTION in Phoenix-Calcite Integration

2016-09-22 Thread Rajeshbabu Chintaguntla (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15512529#comment-15512529 ] Rajeshbabu Chintaguntla commented on PHOENIX-3242: -- Uploaded the patch handling review

[jira] [Commented] (PHOENIX-3242) Support CREATE OR REPLACE FUNCTION in Phoenix-Calcite Integration

2016-09-14 Thread James Taylor (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15490723#comment-15490723 ] James Taylor commented on PHOENIX-3242: --- Good idea, [~julianhyde]. If you agree, let's do it that

[jira] [Commented] (PHOENIX-3242) Support CREATE OR REPLACE FUNCTION in Phoenix-Calcite Integration

2016-09-12 Thread Julian Hyde (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15486350#comment-15486350 ] Julian Hyde commented on PHOENIX-3242: -- [~jamestaylor], Do what Oracle does: never add a new verb,

[jira] [Commented] (PHOENIX-3242) Support CREATE OR REPLACE FUNCTION in Phoenix-Calcite Integration

2016-09-12 Thread James Taylor (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15485869#comment-15485869 ] James Taylor commented on PHOENIX-3242: --- Any advice for us [~julianhyde] for syntax to use for

[jira] [Commented] (PHOENIX-3242) Support CREATE OR REPLACE FUNCTION in Phoenix-Calcite Integration

2016-09-12 Thread Maryann Xue (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15485632#comment-15485632 ] Maryann Xue commented on PHOENIX-3242: -- The patch looks good, [~rajeshbabu]! Thank you very much!

[jira] [Commented] (PHOENIX-3242) Support CREATE OR REPLACE FUNCTION in Phoenix-Calcite Integration

2016-09-06 Thread Rajeshbabu Chintaguntla (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15466953#comment-15466953 ] Rajeshbabu Chintaguntla commented on PHOENIX-3242: -- The reason for the failure is when

[jira] [Commented] (PHOENIX-3242) Support CREATE OR REPLACE FUNCTION in Phoenix-Calcite Integration

2016-09-02 Thread Rajeshbabu Chintaguntla (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15460202#comment-15460202 ] Rajeshbabu Chintaguntla commented on PHOENIX-3242: -- [~maryannxue] The test is not

[jira] [Commented] (PHOENIX-3242) Support CREATE OR REPLACE FUNCTION in Phoenix-Calcite Integration

2016-09-02 Thread Maryann Xue (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15459944#comment-15459944 ] Maryann Xue commented on PHOENIX-3242: -- Thanks for the patch, [~rajeshbabu]! Did the test case