Closed sunshine-Chun closed 1 year ago
@sunshine-Chun has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
@sunshine-Chun has updated the pull request. You must reimport the pull request before landing.
@sunshine-Chun has updated the pull request. You must reimport the pull request before landing.
@sunshine-Chun has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
LGTM (conditionally on the dynamic plugin configuration not broken)
maybe we can add a check ddse is initialized after plugin_register_builtin_and_init_core_se, if not, failed
@sunshine-Chun has updated the pull request. You must reimport the pull request before landing.
@sunshine-Chun has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
@sunshine-Chun has updated the pull request. You must reimport the pull request before landing.
@sunshine-Chun has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
@sunshine-Chun has updated the pull request. You must reimport the pull request before landing.
@sunshine-Chun has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
Early load rocksdb plugin if ddse is rocksdb
Summary:
Test Plan: MTR
Reviewers:
Subscribers:
Tasks:
Tags: