change function signatures

Author: rebasedming

supabase-wrappers/src/instance.rs

@@ -81,6 +81,6 @@ pub(super) unsafe fn create_fdw_instance_from_table_id<
         false => HashMap::new(),
     };
 
-    let wrapper = W::new(&fserver_opts, &user_mapping_opts);
+    let wrapper = W::new(fserver_opts, user_mapping_opts);
     wrapper.report_unwrap()
 }

supabase-wrappers/src/interface.rs

@@ -654,8 +654,8 @@ pub trait ForeignDataWrapper<E: Into<ErrorReport>> {
     /// info or API url in an variable, but don't do heavy works like database
     /// connection or API call.
     fn new(
-        server_options: &HashMap<String, String>,
-        user_mapping_options: &HashMap<String, String>,
+        server_options: HashMap<String, String>,
+        user_mapping_options: HashMap<String, String>,
     ) -> Result<Self, E>
     where
         Self: Sized;
@@ -692,7 +692,7 @@ pub trait ForeignDataWrapper<E: Into<ErrorReport>> {
         columns: &[Column],
         sorts: &[Sort],
         limit: &Option<Limit>,
-        options: &HashMap<String, String>,
+        options: HashMap<String, String>,
     ) -> Result<(), E>;
 
     /// Called when fetch one row from the foreign source

supabase-wrappers/src/options.rs

@@ -95,7 +95,7 @@ pub fn check_options_contain(opt_list: &[Option<String>], tgt: &str) -> Result<(
 }
 
 // convert options definition to hashmap
-pub(super) unsafe fn options_to_hashmap(
+pub unsafe fn options_to_hashmap(
     options: *mut pg_sys::List,
 ) -> Result<HashMap<String, String>, OptionsError> {
     let mut ret = HashMap::new();

supabase-wrappers/src/scan.rs

@@ -87,7 +87,7 @@ impl<E: Into<ErrorReport>, W: ForeignDataWrapper<E>> FdwState<E, W> {
             &self.tgts,
             &self.sorts,
             &self.limit,
-            &self.opts,
+            self.opts.clone(),
         )
     }