Add ORDER BY SEQ_IN_INDEX to MySQL GetIndexes to Fix IndexTests #2152

zeripath · 2022-05-30T10:02:31Z

zeripath commented

2022-05-30 10:02:31 +00:00

MySQL8 appears to return indices in non-SEQ_IN_INDEX order but rather in
column order by default. Adding ORDER BY SEQ_IN_INDEX will result in
index columns being added in the correct order fixing the test.

Signed-off-by: Andrew Thornton art27@cantab.net

MySQL8 appears to return indices in non-SEQ_IN_INDEX order but rather in column order by default. Adding ORDER BY SEQ_IN_INDEX will result in index columns being added in the correct order fixing the test. Signed-off-by: Andrew Thornton <art27@cantab.net>

zeripath added 1 commit 2022-05-30 10:02:32 +00:00

Add ORDER BY SEQ_IN_INDEX to MySQL GetIndexes to Fix IndexTests

MySQL8 appears to return indices in non-SEQ_IN_INDEX order but rather in
column order by default. Adding ORDER BY SEQ_IN_INDEX will result in
index columns being added in the correct order fixing the test.

Signed-off-by: Andrew Thornton <art27@cantab.net>

continuous-integration/drone/pr Build is passing

Details

5adbdfa1ee

lunny approved these changes 2022-05-30 10:09:18 +00:00

lunny merged commit eeb7fcf22c into master

2022-05-30 10:36:23 +00:00

lunny referenced this issue from a commit

2022-05-30 10:36:24 +00:00

Add ORDER BY SEQ_IN_INDEX to MySQL GetIndexes to Fix IndexTests (#2152)

lunny added the

kind

bug

label 2022-05-30 10:36:32 +00:00

lunny added this to the 1.3.1 milestone 2022-05-30 10:36:41 +00:00

Sign in to join this conversation.

No reviewers

No Milestone

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: xorm/xorm#2152