Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

SQL: Generate relevant error message when grouping functions are not used in GROUP BY #38017

Merged
merged 10 commits into from
Feb 2, 2019
Merged

SQL: Generate relevant error message when grouping functions are not used in GROUP BY #38017

merged 10 commits into from
Feb 2, 2019

Commits on Jan 30, 2019

  1. Add checks for Grouping functions restriction to be placed inside GRO… 

    …UP BY
    @astefan
    astefan committed Jan 30, 2019
    Configuration menu
    Copy the full SHA
    2707a44 View commit details
    Browse the repository at this point in the history

Commits on Jan 31, 2019

  1. Fixed bug where GROUP BY HISTOGRAM (not using alias) wasn't recognized 

    properly in the Verifier due to functions equality not working correctly.
    @astefan
    astefan committed Jan 31, 2019
    Configuration menu
    Copy the full SHA
    51b19ef View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' of https://github.com/elastic/elasticsearch int… 

    …o 37952_fix
    @astefan
    astefan committed Jan 31, 2019
    Configuration menu
    Copy the full SHA
    f1f7397 View commit details
    Browse the repository at this point in the history

Commits on Feb 1, 2019

  1. Merge branch 'master' of https://github.com/elastic/elasticsearch int… 

    …o 37952_fix
    @astefan
    astefan committed Feb 1, 2019
    Configuration menu
    Copy the full SHA
    08f75e5 View commit details
    Browse the repository at this point in the history

  2. Address comments

    @astefan
    astefan committed Feb 1, 2019
    Configuration menu
    Copy the full SHA
    2658b70 View commit details
    Browse the repository at this point in the history

  3. Merge branch 'master' of https://github.com/elastic/elasticsearch int… 

    …o 37952_fix
    @astefan
    astefan committed Feb 1, 2019
    Configuration menu
    Copy the full SHA
    36244c7 View commit details
    Browse the repository at this point in the history

  4. Merge branch 'master' of https://github.com/elastic/elasticsearch int… 

    …o 37952_fix
    @astefan
    astefan committed Feb 1, 2019
    Configuration menu
    Copy the full SHA
    ff39950 View commit details
    Browse the repository at this point in the history

Commits on Feb 2, 2019

  1. Changes after merging master

    @astefan
    astefan committed Feb 2, 2019
    Configuration menu
    Copy the full SHA
    08a2bf7 View commit details
    Browse the repository at this point in the history

  2. Address comment

    @astefan
    astefan committed Feb 2, 2019
    Configuration menu
    Copy the full SHA
    597eeaa View commit details
    Browse the repository at this point in the history

  3. Merge branch 'master' of https://github.com/elastic/elasticsearch int… 

    …o 37952_fix
    @astefan
    astefan committed Feb 2, 2019
    Configuration menu
    Copy the full SHA
    0e1b6c4 View commit details
    Browse the repository at this point in the history