-
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
Fix JsonExprBenchmark.cpp #9574
Conversation
This pull request was exported from Phabricator. Differential Revision: D56452087 |
✅ Deploy Preview for meta-velox canceled.
|
Summary: - Replace folly::StringPiece with std::string_view. - Replace `folly::Expected<std::string, bool>` with `std::optional<std::string>`. - Replace private methods isDotKeyFormat and isUnquotedBracketKeyFormat with free functions in .cpp file. Differential Revision: D56451797
Summary: Benchmark used to fail because it didn't initialize memory manager. Differential Revision: D56452087
e13362d
to
0d844c9
Compare
This pull request was exported from Phabricator. Differential Revision: D56452087 |
Summary: Benchmark used to fail because it didn't initialize memory manager. Differential Revision: D56452087
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.
@mbasmanova thanks!
Summary: Benchmark used to fail because it didn't initialize memory manager. Reviewed By: xiaoxmeng Differential Revision: D56452087
Summary: Benchmark used to fail because it didn't initialize memory manager. Reviewed By: xiaoxmeng Differential Revision: D56452087
This pull request has been merged in fb9355d. |
Conbench analyzed the 1 benchmark run on commit There was 1 benchmark result indicating a performance regression:
The full Conbench report has more details. |
Summary: Pull Request resolved: facebookincubator#9574 Benchmark used to fail because it didn't initialize memory manager. Reviewed By: xiaoxmeng Differential Revision: D56452087 fbshipit-source-id: b564b457728b0efff49e2b72c3aed5aa99dedd1d
Summary: Benchmark used to fail because it didn't initialize memory manager.
Differential Revision: D56452087