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

Jump to bottom

add test for PI() #8

Closed
wants to merge 3 commits into from
Closed

add test for PI() #8

wants to merge 3 commits into from

Conversation

Fullstop000
Copy link
Member

@Fullstop000 Fullstop000 commented Nov 6, 2019 •
edited

Add test case for PI()

Signed-off-by: Fullstop000 fullstop1005@gmail.com

Relate to tikv/tikv#5812

@lonng

@Fullstop000
Copy link
Member Author

Tested in local env

2019/11/07 09:57:41 Test summary: Test case PASS
2019/11/07 09:57:41 Testing [sql/math.sql]...
2019/11/07 09:57:41 Parsing...
2019/11/07 09:57:41 Running...
2019/11/07 09:57:41 Info: run sql
SELECT PI();
2019/11/07 09:57:41 Test summary: Success queries: 1, fail (and ignore) queries: 0 (where there are 0 non-matching queries)

@lonng
Copy link
Member

lonng commented Nov 12, 2019

@Fullstop000 Please resolve the conflicts.

@Fullstop000
add test for PI()
72b62a7 
Signed-off-by: Fullstop000 <fullstop1005@gmail.com>
@Fullstop000
resolve conflict
5832d28 
Signed-off-by: Fullstop000 <fullstop1005@gmail.com>
@Fullstop000
remove math.sql
d12ed57 
Signed-off-by: Fullstop000 <fullstop1005@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Fullstop000 @lonng