You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2021/10/08 16:47:19 UTC

[GitHub] [pinot] elonazoulay opened a new issue #7545: Column alias with same name as column causes pinot to hang

elonazoulay opened a new issue #7545:
URL: https://github.com/apache/pinot/issues/7545


   If you have a function on a column and then alias that to the same column name pinot hangs.
   To reproduce start the quick start batch and run the following query:
   ```
   select cast(runs as long) as runs from baseballStats limit 10
   ```
   
   This query works:
   ```
   select cast(runs as long) as runs2 from baseballStats limit 10
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


[GitHub] [pinot] SandishKumarHN commented on issue #7545: Column alias with same name as column causes pinot to hang

Posted by GitBox <gi...@apache.org>.
SandishKumarHN commented on issue #7545:
URL: https://github.com/apache/pinot/issues/7545#issuecomment-940232814


   working on it


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


[GitHub] [pinot] SandishKumarHN commented on issue #7545: Column alias with same name as column causes pinot to hang

Posted by GitBox <gi...@apache.org>.
SandishKumarHN commented on issue #7545:
URL: https://github.com/apache/pinot/issues/7545#issuecomment-941146912


   @elonazoulay looks like it doesn't hang for me. it throws a proper error message.  
   ```
   select cast(runs as long) as runs from baseballStats limit 10
   ```
   ```
   org.apache.pinot.sql.parsers.SqlCompilationException: Alias runs cannot be referred in SELECT Clause
   	at org.apache.pinot.sql.parsers.CalciteSqlParser.matchIdentifierInAliasMap(CalciteSqlParser.java:156)
   	at org.apache.pinot.sql.parsers.CalciteSqlParser.matchIdentifierInAliasMap(CalciteSqlParser.java:148)
   	at org.apache.pinot.sql.parsers.CalciteSqlParser.matchIdentifierInAliasMap(CalciteSqlParser.java:144)
   	at org.apache.pinot.sql.parsers.CalciteSqlParser.validateSelectionClause(CalciteSqlParser.java:135))
   ```
   please correct me, if I'm missing anything here. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


[GitHub] [pinot] xiangfu0 commented on issue #7545: Column alias with same name as column causes pinot to hang

Posted by GitBox <gi...@apache.org>.
xiangfu0 commented on issue #7545:
URL: https://github.com/apache/pinot/issues/7545#issuecomment-939148889


   Thanks for reporting this. I think this is a bug inside the alias check. Will fix it.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


[GitHub] [pinot] Jackie-Jiang commented on issue #7545: Column alias with same name as column causes pinot to hang

Posted by GitBox <gi...@apache.org>.
Jackie-Jiang commented on issue #7545:
URL: https://github.com/apache/pinot/issues/7545#issuecomment-942853931


   @SandishKumarHN IIRC, there was a bug that causes the query console to hang when the query fails the compilation, and it has been fixed and can return the error message properly. The goal for this issue is to fix the error.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


[GitHub] [pinot] SandishKumarHN commented on issue #7545: Column alias with same name as column causes pinot to hang

Posted by GitBox <gi...@apache.org>.
SandishKumarHN commented on issue #7545:
URL: https://github.com/apache/pinot/issues/7545#issuecomment-955895523


   this issue is fixed as part of this PR https://github.com/apache/pinot/pull/7576/, just verified it. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


[GitHub] [pinot] Jackie-Jiang commented on issue #7545: Column alias with same name as column causes pinot to hang

Posted by GitBox <gi...@apache.org>.
Jackie-Jiang commented on issue #7545:
URL: https://github.com/apache/pinot/issues/7545#issuecomment-955896871


   @xiangfu0 @elonazoulay Can you please verify the fix and close the issue if the issue is fixed?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org


[GitHub] [pinot] SandishKumarHN commented on issue #7545: Column alias with same name as column causes pinot to hang

Posted by GitBox <gi...@apache.org>.
SandishKumarHN commented on issue #7545:
URL: https://github.com/apache/pinot/issues/7545#issuecomment-941146912


   @elonazoulay looks like it doesn't hang for me. it throws a proper error message.  
   ```
   select cast(runs as long) as runs from baseballStats limit 10
   ```
   ```
   org.apache.pinot.sql.parsers.SqlCompilationException: Alias runs cannot be referred in SELECT Clause
   	at org.apache.pinot.sql.parsers.CalciteSqlParser.matchIdentifierInAliasMap(CalciteSqlParser.java:156)
   	at org.apache.pinot.sql.parsers.CalciteSqlParser.matchIdentifierInAliasMap(CalciteSqlParser.java:148)
   	at org.apache.pinot.sql.parsers.CalciteSqlParser.matchIdentifierInAliasMap(CalciteSqlParser.java:144)
   	at org.apache.pinot.sql.parsers.CalciteSqlParser.validateSelectionClause(CalciteSqlParser.java:135))
   ```
   please correct me, if I'm missing anything here. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org