[CALCITE-7404] Incorrect Field Alias in MongoDB project Stage #4774
Conversation
cjj2010
changed the title
cjj2010
changed the title
cjj2010
changed the title
cjj2010
changed the title
|
Jira mentioned some issues that need to be confirmed first. |
| * Incorrect Field Alias in MongoDB project Stage</a>. */ | ||
| @Test void testAggFunctionMinFilter() { | ||
| assertModel(MODEL) | ||
| .query("select min(pop>5000) as pop_result from zips") |
There was a problem hiding this comment.
Could you add more tests? For example, nested aliases, such as (a as b) as c.
There was a problem hiding this comment.
Could you add more tests? For example, nested aliases, such as
(a as b) as c.
I want to ask if it's similar "select (pop as popp) as pop_result from zips" ?
There was a problem hiding this comment.
Yes, does this case meet expectations?
There was a problem hiding this comment.
is case meet expectations?
Errors will occur during the syntax parsing phase of SQL,
but: was "java.sql.SQLException: Error while executing SQL "select (pop as pop_a) as pop_result from zips": parse failed: Encountered "as" at line 1, column 13.
Was expecting one of:
"EXCEPT" ...
"FETCH" ...
There was a problem hiding this comment.
I tested it, and aliases don't support nesting, so this issue can be ignored.
I think can also add some general tests, such as those that don't include aggregate functions, like select pop as popp from zips.
There was a problem hiding this comment.
I tested it, and aliases don't support nesting, so this issue can be ignored. I think can also add some general tests, such as those that don't include aggregate functions, like
select pop as popp from zips.
Okay, I have added it
There was a problem hiding this comment.
I think it's fine to just put cases in a test method.
2 participants
jira: https://issues.apache.org/jira/projects/CALCITE/issues/CALCITE-7404