-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Extract fuzzer into expression/fuzzer directory #9651
Conversation
✅ Deploy Preview for meta-velox canceled.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rui-mo This is nice. Thank you for refactoring, Rui.
@kagamiori has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Hi @rui-mo, could you rebase onto the latest main? There is a merge conflict in ExpressionFuzzerVerifier.h. Thanks! |
@kagamiori This PR was rebased. Thanks for the reminder. |
Hi @rui-mo, sorry that there is a new merge conflict again while I ran some tests. Could you rebase again? I'll try to land as soon as possible before the next merge conflict. |
@kagamiori This PR was rebased. Thanks for your help. |
@kagamiori merged this pull request in 3f3895d. |
Conbench analyzed the 1 benchmark run on commit There were no benchmark performance regressions. 🎉 The full Conbench report has more details. |
Summary: Extract fuzzer from expression/tests into expression/fuzzer directory. Pull Request resolved: facebookincubator#9651 Reviewed By: mbasmanova Differential Revision: D56710670 Pulled By: kagamiori fbshipit-source-id: aec746acfc8de97263fa840012ab1ea51b681ca7
Extract fuzzer from expression/tests into expression/fuzzer directory.