Skip to content

Conversation

@xudong963
Copy link
Member

Rationale for this change

I noticed the two tests would be unstable somethings.

What changes are included in this PR?

Add Order by to make them stable.

Are these changes tested?

Yes

Are there any user-facing changes?

No

@github-actions github-actions bot added the sqllogictest SQL Logic Tests (.slt) label Jan 17, 2025
Copy link
Contributor

@comphead comphead left a comment

Choose a reason for hiding this comment

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

lgtm thanks @xudong963

@xudong963 xudong963 merged commit 3082ac6 into apache:main Jan 17, 2025
25 checks passed
@xudong963 xudong963 deleted the fix_flaky_test branch January 17, 2025 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sqllogictest SQL Logic Tests (.slt)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants