[jira] [Commented] (PHOENIX-3263) Allow comma before CONSTRAINT to be optional

2016-09-13 Thread Eric Lomore (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15487831#comment-15487831 ] Eric Lomore commented on PHOENIX-3263: -- At the moment, Phoenix supports both comma

[jira] [Commented] (PHOENIX-3263) Allow comma before CONSTRAINT to be optional

2016-09-13 Thread James Taylor (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15487914#comment-15487914 ] James Taylor commented on PHOENIX-3263: --- [~lomoree] - I don't completely understan

[jira] [Commented] (PHOENIX-3263) Allow comma before CONSTRAINT to be optional

2016-09-13 Thread Maryann Xue (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15487974#comment-15487974 ] Maryann Xue commented on PHOENIX-3263: -- Hey, [~lomoree], in Calcite-Phoenix we are

[jira] [Commented] (PHOENIX-3263) Allow comma before CONSTRAINT to be optional

2016-09-13 Thread Eric Lomore (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15488400#comment-15488400 ] Eric Lomore commented on PHOENIX-3263: -- Thanks! This makes a lot more sense now, st

[jira] [Commented] (PHOENIX-3263) Allow comma before CONSTRAINT to be optional

2016-09-13 Thread Maryann Xue (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15488424#comment-15488424 ] Maryann Xue commented on PHOENIX-3263: -- Thanks for you response, [~lomoree]! But yo

[jira] [Commented] (PHOENIX-3263) Allow comma before CONSTRAINT to be optional

2016-09-22 Thread James Taylor (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15513709#comment-15513709 ] James Taylor commented on PHOENIX-3263: --- [~lomoree] - would you mind attaching a p

[jira] [Commented] (PHOENIX-3263) Allow comma before CONSTRAINT to be optional

2016-09-22 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15514535#comment-15514535 ] Hadoop QA commented on PHOENIX-3263: {color:red}-1 overall{color}. Here are the res

[jira] [Commented] (PHOENIX-3263) Allow comma before CONSTRAINT to be optional

2016-09-22 Thread James Taylor (JIRA)
[ https://issues.apache.org/jira/browse/PHOENIX-3263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15514561#comment-15514561 ] James Taylor commented on PHOENIX-3263: --- +1. Thanks, [~lomoree]. I committed this