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.

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

[Examiner] fix mysql 5.7 error -- add e.examinerID into group by #2872

Merged
merged 1 commit into from Jun 9, 2017

Conversation

kongtiaowang
Copy link
Contributor

This pull request adding e.examinerID into "group by" clause.
Fix the error in Mysql 5.7.
[Expression #2 of SELECT list is not in GROUP BY clause and contains nonaggregated column 'loris_test.e.examinerID' which is not functionally dependent on columns in GROUP BY clause; this is incompatible with sql_mode=only_full_group_by]

@kongtiaowang kongtiaowang requested a review from driusan June 9, 2017 16:51
@kongtiaowang kongtiaowang added the Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) label Jun 9, 2017
@kongtiaowang kongtiaowang added this to the 17.0 milestone Jun 9, 2017
Copy link
Collaborator

@driusan driusan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same code is already in 17.1, this is just a backport for the user who complained about getting the error in 17.0.

@driusan driusan merged commit f7f67d8 into aces:17.0-dev Jun 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants