refactor: change plugin option type from &[PluginOptions] to Option<&PluginOptions> for understandability (#6763)

refactor: change plugin option type from `[PluginOptions]` to `Option<&PluginOptions>` for understandability

Signed-off-by: zyy17 <zyylsxm@gmail.com>
This commit is contained in:
zyy17
2025-08-19 19:11:00 -07:00
committed by GitHub
parent 7af471c5aa
commit 5420d6f7fb
11 changed files with 15 additions and 15 deletions

View File

@@ -22,7 +22,7 @@ use crate::options::PluginOptions;
#[allow(unused_mut)]
pub async fn setup_datanode_plugins(
plugins: &mut Plugins,
plugin_options: &[PluginOptions],
plugin_options: Option<&PluginOptions>,
dn_opts: &DatanodeOptions,
) -> Result<()> {
Ok(())

View File

@@ -21,7 +21,7 @@ use crate::options::PluginOptions;
#[allow(unused_mut)]
pub async fn setup_flownode_plugins(
_plugins: &mut Plugins,
_plugin_options: &[PluginOptions],
_plugin_options: Option<&PluginOptions>,
_fn_opts: &FlownodeOptions,
) -> Result<()> {
Ok(())

View File

@@ -23,7 +23,7 @@ use crate::options::PluginOptions;
#[allow(unused_mut)]
pub async fn setup_frontend_plugins(
plugins: &mut Plugins,
_plugin_options: &[PluginOptions],
_plugin_options: Option<&PluginOptions>,
fe_opts: &FrontendOptions,
) -> Result<()> {
if let Some(user_provider) = fe_opts.user_provider.as_ref() {

View File

@@ -21,7 +21,7 @@ use crate::options::PluginOptions;
#[allow(unused_variables)]
pub async fn setup_metasrv_plugins(
_plugins: &mut Plugins,
plugin_options: &[PluginOptions],
_plugin_options: Option<&PluginOptions>,
metasrv_opts: &MetasrvOptions,
) -> Result<()> {
Ok(())

View File

@@ -18,6 +18,6 @@ use serde::{Deserialize, Serialize};
pub struct DummyOptions;
#[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]
pub enum PluginOptions {
Dummy(DummyOptions),
pub struct PluginOptions {
pub dummy: DummyOptions,
}