Skip to content

expression: implement vectorized evaluation for builtinAsinSig#12222

Merged
zz-jason merged 4 commits intopingcap:masterfrom
jacklightChen:asin
Sep 17, 2019
Merged

expression: implement vectorized evaluation for builtinAsinSig#12222
zz-jason merged 4 commits intopingcap:masterfrom
jacklightChen:asin

Conversation

@jacklightChen
Copy link
Contributor

What problem does this PR solve?

implement vectorized evaluation for builtinAsinSig, for #12105

What is changed and how it works?

according to benchmark, about 2 times faster than before:

BenchmarkVectorizedBuiltinMathFunc/builtinAsinSig-VecBuiltinFunc-8                100000             16326 ns/op            0 B/op          0 allocs/op
BenchmarkVectorizedBuiltinMathFunc/builtinAsinSig-NonVecBuiltinFunc-8              50000             32405 ns/op            0 B/op          0 allocs/op

Check List

Tests

  • Unit test

@sre-bot sre-bot added the contribution This PR is from a community contributor. label Sep 17, 2019
@codecov
Copy link

codecov bot commented Sep 17, 2019

Codecov Report

Merging #12222 into master will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master     #12222   +/-   ##
===========================================
  Coverage   81.0969%   81.0969%           
===========================================
  Files           454        454           
  Lines         98524      98524           
===========================================
  Hits          79900      79900           
  Misses        12859      12859           
  Partials       5765       5765

Copy link
Contributor

@Reminiscent Reminiscent left a comment

Choose a reason for hiding this comment

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

LGTM

@Reminiscent Reminiscent added the status/LGT1 Indicates that a PR has LGTM 1. label Sep 17, 2019
qw4990
qw4990 previously approved these changes Sep 17, 2019
Copy link
Contributor

@qw4990 qw4990 left a comment

Choose a reason for hiding this comment

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

LGTM

@qw4990 qw4990 added status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Sep 17, 2019
@sre-bot
Copy link
Contributor

sre-bot commented Sep 17, 2019

Your auto merge job has been accepted, waiting for 12221

@qw4990
Copy link
Contributor

qw4990 commented Sep 17, 2019

@jacklightChen Please resolve conflicts... again 🤣

@qw4990 qw4990 added status/can-merge Indicates a PR has been approved by a committer. and removed status/can-merge Indicates a PR has been approved by a committer. labels Sep 17, 2019
@sre-bot
Copy link
Contributor

sre-bot commented Sep 17, 2019

/run-all-tests

@sre-bot
Copy link
Contributor

sre-bot commented Sep 17, 2019

@jacklightChen merge failed.

@jacklightChen
Copy link
Contributor Author

/run-all-tests

@qw4990
Copy link
Contributor

qw4990 commented Sep 17, 2019

/run-all-tests

Copy link
Member

@zz-jason zz-jason left a comment

Choose a reason for hiding this comment

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

LGTM

@zz-jason zz-jason merged commit 3e1409a into pingcap:master Sep 17, 2019
@zz-jason zz-jason added status/all tests passed and removed status/can-merge Indicates a PR has been approved by a committer. labels Sep 17, 2019
jacklightChen added a commit to jacklightChen/tidb that referenced this pull request Sep 17, 2019
expression: implement vectorized evaluation for builtinAsinSig (pingcap#12222)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/expression contribution This PR is from a community contributor. status/LGT2 Indicates that a PR has LGTM 2.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants