velox
velox copied to clipboard
Add per-test memory sanity check in OperatorTestBase
This per test sanity check allows test to reveal failures on test level instead of on the entire suite level. This will greatly reduce the time engineers spent in locating failed tests due to memory leak.
Deploy Preview for meta-velox canceled.
Name | Link |
---|---|
Latest commit | 022590f767612d4c85e63415166ccbd554ff311a |
Latest deploy log | https://app.netlify.com/sites/meta-velox/deploys/6615826c8f90cf0008e41213 |
@tanjialiang has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
@tanjialiang has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
@tanjialiang has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
@tanjialiang has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
@tanjialiang has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
@tanjialiang has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
@tanjialiang has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
This pull request was exported from Phabricator. Differential Revision: D55043514
@tanjialiang merged this pull request in facebookincubator/velox@845f13ca7827fc9383cd069d541b55fe61f94dcd.
Conbench analyzed the 1 benchmark run on commit 845f13ca
.
There were no benchmark performance regressions. 🎉
The full Conbench report has more details.