search

facebookincubator / velox

A composable and fully extensible C++ execution engine library for data management systems.
https://velox-lib.io/
Apache License 2.0
3.54k stars 1.17k forks source link

fix: Init filesystem after registered in query replyaer #11559

Closed duanmeng closed 1 week ago

duanmeng commented 1 week ago

Get a trace dir filesystem after filesystems are registered during query replayer initialization.

netlify[bot] commented 1 week ago

Deploy Preview for meta-velox canceled.

Name Link
Latest commit 0ac89748f140b6cde7f4fba47e7f2853b9409140
Latest deploy log https://app.netlify.com/sites/meta-velox/deploys/6738203a6b58d00008704ab7
facebook-github-bot commented 1 week ago

@xiaoxmeng has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

facebook-github-bot commented 1 week ago

@xiaoxmeng merged this pull request in facebookincubator/velox@d1ce7195b927881b3a71f96a6056be857146f251.

conbench-facebook[bot] commented 1 week ago

Conbench analyzed the 1 benchmark run on commit d1ce7195.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.