You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by GitBox <gi...@apache.org> on 2021/11/04 12:25:41 UTC

[GitHub] [calcite] ILuffZhe opened a new pull request #2599: [Calcite 4868] Support sorting aggregation result in Elasticsearch Adapter

ILuffZhe opened a new pull request #2599:
URL: https://github.com/apache/calcite/pull/2599


   The PR supports sorting aggregation result in Elasticsearch Adapter.
   Like: **select val2, max(val3) as MAX_VAL3 from view group by val2 order by MAX_VAL3 desc**


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

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



[GitHub] [calcite] ILuffZhe commented on pull request #2599: [CALCITE-4868] Elasticsearch adapter fails if GROUP BY is followed by ORDER BY

Posted by GitBox <gi...@apache.org>.
ILuffZhe commented on pull request #2599:
URL: https://github.com/apache/calcite/pull/2599#issuecomment-997128577


   Hi, @NobiGo . I've add some description for this fix, please take a look when you are free.


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

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



[GitHub] [calcite] ILuffZhe commented on pull request #2599: [CALCITE-4868] Elasticsearch adapter fails if GROUP BY is followed by ORDER BY

Posted by GitBox <gi...@apache.org>.
ILuffZhe commented on pull request #2599:
URL: https://github.com/apache/calcite/pull/2599#issuecomment-999555919


   I'll reopen this PR after https://github.com/apache/calcite/pull/2657 since those two are highly related.


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

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



[GitHub] [calcite] NobiGo commented on pull request #2599: [Calcite 4868] Support sorting aggregation result in Elasticsearch Adapter

Posted by GitBox <gi...@apache.org>.
NobiGo commented on pull request #2599:
URL: https://github.com/apache/calcite/pull/2599#issuecomment-961495783


   1) Please reformat the commit info like[CALCITE-XXXX] XXXXXXXXX. 
   2)The commit info needs to stay the same as The Jira's summary.


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

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



[GitHub] [calcite] NobiGo commented on pull request #2599: [Calcite 4868] Support sorting aggregation result in Elasticsearch Adapter

Posted by GitBox <gi...@apache.org>.
NobiGo commented on pull request #2599:
URL: https://github.com/apache/calcite/pull/2599#issuecomment-961495783


   1) Please reformat the commit info like[CALCITE-XXXX] XXXXXXXXX. 
   2)The commit info needs to stay the same as The Jira's summary.


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

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



[GitHub] [calcite] NobiGo commented on pull request #2599: [Calcite 4868] Support sorting aggregation result in Elasticsearch Adapter

Posted by GitBox <gi...@apache.org>.
NobiGo commented on pull request #2599:
URL: https://github.com/apache/calcite/pull/2599#issuecomment-961495783


   1) Please reformat the commit info like[CALCITE-XXXX] XXXXXXXXX. 
   2)The commit info needs to stay the same as The Jira's summary.


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

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