megakaizo opened a new pull request, #23139: URL: https://github.com/apache/datafusion/pull/23139
### Which issue does this PR close? - Part of #23080 ### Rationale for this change `DiskManagerConfig` and `DiskManager::try_new` were deprecated in version 48.0.0 in favor of the new `DiskManagerBuilder`. This PR remove this deprecated methods and update Runtime env initialization logic ### What changes are included in this PR? - Removed `RuntimeEnvBuilder::with_disk_manager` depercated method. - Removed deprecated `DiskManagerConfig` enum and its associated constructor methods. - Removed deprecated `DiskManager::try_new` method. - Refactored `RuntimeEnvBuilder` to store `Option<Arc<DiskManager>>` instead of the old configuration enum. - Update `RuntimeEnvBuilder::build` logic by using matching to handle the existing manager, builder, or default initialization. ### Are these changes tested? verifed by running local tests ### Are there any user-facing changes? Yes. This removes the deprecated public Rust APIs `RuntimeEnvBuilder::with_disk_manager`, `DiskManagerConfig` and `DiskManager::try_new`. Downstream users who need to configure the disk manager should migrate to using `DiskManager::builder`. This is an API change and should be labeled as `api change`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
