You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2022/09/30 17:32:49 UTC

[GitHub] [druid] 7Luma opened a new issue, #13163: Join queries with order by clause doesn't work

7Luma opened a new issue, #13163:
URL: https://github.com/apache/druid/issues/13163

   Hi!
   Join queries with order by clause doesn't work
   
   ### Affected Version
   
   0.23.0, 0.24.0
   
   ### Description
   
   Queries like (with join and order by clause):
   ```
   SELECT "T1"."__time"
   FROM "SomeTableA" AS "T1"
   JOIN "SomeTableB" AS "T2"
   ON "T1"."A" = "T2"."B"
   ORDER BY "T1"."__time" ASC
   ```
   Doesn't work at 0.23.0 and 0.24.0 versions of Druid (but works at 0.22.0).
   "Doesn't work" means that query processing ends with exception on broker node:
   ```
   Error: Unknown exception
   Cannot convert query parts into an actual query
   org.apache.druid.sql.calcite.rel.CannotBuildQueryException
   ```
   More detailed log here:
   [broker.log](https://github.com/apache/druid/files/9686541/broker.log)
   


-- 
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@druid.apache.org.apache.org

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


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


[GitHub] [druid] gianm commented on issue #13163: Join queries with order by clause doesn't work

Posted by GitBox <gi...@apache.org>.
gianm commented on issue #13163:
URL: https://github.com/apache/druid/issues/13163#issuecomment-1269239299

   I haven't had a chance to write a unit test exhibiting this problem yet. If you have one that would certainly accelerate the process, and be very appreciated! It would go in CalciteJoinQueryTest. If not, that's ok, we will come around to do it on our own.


-- 
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@druid.apache.org

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


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


[GitHub] [druid] gianm commented on issue #13163: Join queries with order by clause doesn't work

Posted by GitBox <gi...@apache.org>.
gianm commented on issue #13163:
URL: https://github.com/apache/druid/issues/13163#issuecomment-1270560685

   Thanks for looking into it @somu-imply!


-- 
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@druid.apache.org

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


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


[GitHub] [druid] abhishekagarwal87 commented on issue #13163: Join queries with order by clause doesn't work

Posted by GitBox <gi...@apache.org>.
abhishekagarwal87 commented on issue #13163:
URL: https://github.com/apache/druid/issues/13163#issuecomment-1269780747

   I think https://github.com/apache/druid/pull/12418 caused it. 
   @somu-imply FYI.


-- 
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@druid.apache.org

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


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


[GitHub] [druid] somu-imply commented on issue #13163: Join queries with order by clause doesn't work

Posted by GitBox <gi...@apache.org>.
somu-imply commented on issue #13163:
URL: https://github.com/apache/druid/issues/13163#issuecomment-1270473090

   I'll add the unit test. Seems like #13173 fixes this. The query under consideration is 
   ```
   select t1.__time 
   from foo as t1
   JOIN numFoo as t2
   on t1.dim2 = t2.dim2
   order by t1.__time ASC
   ```
   
   I'll update the unit test case with this query


-- 
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@druid.apache.org

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


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


[GitHub] [druid] somu-imply commented on issue #13163: Join queries with order by clause doesn't work

Posted by GitBox <gi...@apache.org>.
somu-imply commented on issue #13163:
URL: https://github.com/apache/druid/issues/13163#issuecomment-1270822431

   The test case has been added through #13173 


-- 
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@druid.apache.org

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


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