[jira] [Commented] (CALCITE-2102) SqlToRelConverter (or perhaps RelBuilder) ignore repeat sort keys. In addition, it should also prevent the incompatible sort keys.

2017-12-20 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2102?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16298075#comment-16298075 ] John Fang commented on CALCITE-2102: [~julianhyde] If the sql contains the incompatible sort keys,

[jira] [Updated] (CALCITE-2102) RelCollation should have a constraint to limit the repeat or incompatible RelFieldCollations.

2017-12-19 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2102?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] John Fang updated CALCITE-2102: --- Description: I test the query {{SELECT a, c FROM MyTable ORDER BY a DESC, a DESC}}. And the

[jira] [Updated] (CALCITE-2102) RelCollation should have a constraint to limit the repeat or incompatible RelFieldCollations.

2017-12-19 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2102?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] John Fang updated CALCITE-2102: --- Description: I test the query ```SELECT a, c FROM MyTable ORDER BY a DESC, a DESC```. And

[jira] [Updated] (CALCITE-2102) RelCollation should have a constraint to limit the repeat or incompatible RelFieldCollations.

2017-12-19 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2102?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] John Fang updated CALCITE-2102: --- Description: I test the query `SELECT a, c FROM MyTable ORDER BY a DESC, a DESC`. And the

[jira] [Created] (CALCITE-2102) RelCollation should have a constraint to limit the repeat or incompatible RelFieldCollations.

2017-12-19 Thread John Fang (JIRA)
John Fang created CALCITE-2102: -- Summary: RelCollation should have a constraint to limit the repeat or incompatible RelFieldCollations. Key: CALCITE-2102 URL: https://issues.apache.org/jira/browse/CALCITE-2102

[jira] [Updated] (CALCITE-2102) RelCollation should have a constraint to limit the repeat or incompatible RelFieldCollations.

2017-12-19 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2102?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] John Fang updated CALCITE-2102: --- Description: I test the query {{SELECT a, c FROM MyTable ORDER BY a DESC, a DESC}}. And

[jira] [Updated] (CALCITE-2102) RelCollation should have a constraint to limit the repeat or incompatible RelFieldCollations.

2017-12-19 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2102?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] John Fang updated CALCITE-2102: --- Description: I test the query {{SELECT a, c FROM MyTable ORDER BY a DESC, a DESC}}. And the

[jira] [Updated] (CALCITE-2102) SqlToRelConverter (or perhaps RelBuilder) ignore repeat sort keys. In addition, it should also prevent the incompatible sort keys.

2017-12-19 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2102?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] John Fang updated CALCITE-2102: --- Summary: SqlToRelConverter (or perhaps RelBuilder) ignore repeat sort keys. In addition, it should

[jira] [Commented] (CALCITE-2102) SqlToRelConverter (or perhaps RelBuilder) ignore repeat sort keys. In addition, it should also prevent the incompatible sort keys.

2017-12-19 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2102?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16297975#comment-16297975 ] John Fang commented on CALCITE-2102: PR sent https://github.com/apache/calcite/pull/587 >

[jira] [Updated] (CALCITE-2323) The configuration parameter "defaultNullCollation" which enum NullCollation { LOW, HIGH, FIRST, LAST } to allow alternative policies, but it can't take effect for over(

2018-05-23 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2323?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] John Fang updated CALCITE-2323: --- Description: The configuration parameter "defaultNullCollation"  can't take effect for the query

[jira] [Updated] (CALCITE-2323) The configuration parameter "defaultNullCollation" which enum NullCollation { LOW, HIGH, FIRST, LAST } to allow alternative policies, but it can't take effect for over(

2018-05-23 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2323?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] John Fang updated CALCITE-2323: --- Description: For example, the following query: *select deptno, rank() over(partition by empno

[jira] [Commented] (CALCITE-2323) The configuration parameter "defaultNullCollation" which enum NullCollation { LOW, HIGH, FIRST, LAST } to allow alternative policies, but it can't take effect for ove

2018-05-24 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2323?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16490169#comment-16490169 ] John Fang commented on CALCITE-2323: Thanks for [~julianhyde]. I I have update the PR according to

[jira] [Updated] (CALCITE-2323) The configuration parameter "defaultNullCollation" which enum NullCollation { LOW, HIGH, FIRST, LAST } to allow alternative policies, but it can't take effect for over(

2018-05-23 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2323?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] John Fang updated CALCITE-2323: --- Description: The query with over can't  > The configuration parameter "defaultNullCollation" which

[jira] [Updated] (CALCITE-2323) The configuration parameter "defaultNullCollation" which enum NullCollation { LOW, HIGH, FIRST, LAST } to allow alternative policies, but it can't take effect for over(

2018-05-23 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2323?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] John Fang updated CALCITE-2323: --- Description: [The query with over can't  |https://github.com/apache/calcite/pull/697] (was: The

[jira] [Comment Edited] (CALCITE-2323) The configuration parameter "defaultNullCollation" which enum NullCollation { LOW, HIGH, FIRST, LAST } to allow alternative policies, but it can't take effect fo

2018-05-23 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2323?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16487563#comment-16487563 ] John Fang edited comment on CALCITE-2323 at 5/23/18 4:20 PM: - The PR is .

[jira] [Updated] (CALCITE-2323) The configuration parameter "defaultNullCollation" which enum NullCollation { LOW, HIGH, FIRST, LAST } to allow alternative policies, but it can't take effect for over(

2018-05-23 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2323?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] John Fang updated CALCITE-2323: --- Description: The configuration parameter "defaultNullCollation"  can't take effect for the query

[jira] [Created] (CALCITE-2323) The configuration parameter "defaultNullCollation" which enum NullCollation { LOW, HIGH, FIRST, LAST } to allow alternative policies, but it can't take effect for over(

2018-05-23 Thread John Fang (JIRA)
John Fang created CALCITE-2323: -- Summary: The configuration parameter "defaultNullCollation" which enum NullCollation { LOW, HIGH, FIRST, LAST } to allow alternative policies, but it can't take effect for over(); Key: CALCITE-2323

[jira] [Commented] (CALCITE-2323) The configuration parameter "defaultNullCollation" which enum NullCollation { LOW, HIGH, FIRST, LAST } to allow alternative policies, but it can't take effect for ove

2018-05-23 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2323?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16487563#comment-16487563 ] John Fang commented on CALCITE-2323: the PR is . https://github.com/apache/calcite/pull/697 > The

[jira] [Updated] (CALCITE-2323) The configuration parameter "defaultNullCollation" which enum NullCollation { LOW, HIGH, FIRST, LAST } to allow alternative policies, but it can't take effect for over(

2018-05-23 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2323?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] John Fang updated CALCITE-2323: --- Description: The configuration parameter "defaultNullCollation"  can't take effect for the query

[jira] [Updated] (CALCITE-2323) The configuration parameter "defaultNullCollation" which enum NullCollation { LOW, HIGH, FIRST, LAST } to allow alternative policies, but it can't take effect for over(

2018-05-23 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2323?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] John Fang updated CALCITE-2323: --- Description: The configuration parameter "defaultNullCollation"  can't take effect for the query

[jira] [Commented] (CALCITE-2323) The configuration parameter "defaultNullCollation" which enum NullCollation { LOW, HIGH, FIRST, LAST } to allow alternative policies, but it can't take effect for ove

2018-06-05 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2323?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16502797#comment-16502797 ] John Fang commented on CALCITE-2323: [~julianhyde] can you help me review it again? > The

[jira] [Comment Edited] (CALCITE-2102) SqlToRelConverter (or perhaps RelBuilder) ignore repeat sort keys. In addition, it should also prevent the incompatible sort keys.

2017-12-21 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2102?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16300040#comment-16300040 ] John Fang edited comment on CALCITE-2102 at 12/21/17 1:54 PM: --

[jira] [Commented] (CALCITE-2102) SqlToRelConverter (or perhaps RelBuilder) ignore repeat sort keys. In addition, it should also prevent the incompatible sort keys.

2017-12-21 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2102?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16300040#comment-16300040 ] John Fang commented on CALCITE-2102: [~julianhyde] Thank you. I have already add a unit test in

[jira] [Commented] (CALCITE-2102) SqlToRelConverter (or perhaps RelBuilder) ignore repeat sort keys. In addition, it should also prevent the incompatible sort keys.

2017-12-21 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2102?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16300047#comment-16300047 ] John Fang commented on CALCITE-2102: You can run the unit test

[jira] [Updated] (CALCITE-2102) ignore repeat sort keys

2017-12-21 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2102?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] John Fang updated CALCITE-2102: --- Summary: ignore repeat sort keys (was: SqlToRelConverter (or perhaps RelBuilder) ignore repeat

[jira] [Commented] (CALCITE-2102) ignore repeat sort keys

2017-12-21 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2102?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16300092#comment-16300092 ] John Fang commented on CALCITE-2102: [~julianhyde] your branch's {{SqlToRelConverterTest.testOrder}}

[jira] [Commented] (CALCITE-2102) SqlToRelConverter (or perhaps RelBuilder) ignore repeat sort keys. In addition, it should also prevent the incompatible sort keys.

2017-12-20 Thread John Fang (JIRA)
[ https://issues.apache.org/jira/browse/CALCITE-2102?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16299376#comment-16299376 ] John Fang commented on CALCITE-2102: Got it. Thank you. > SqlToRelConverter (or perhaps RelBuilder)