Change parameters from String to &str (#43)

## Problem

One of the parameters for `fetch` was `&[String]` instead of `&[&str]`.
Similarly, one of the parameters for `query_by_id` was `String` instead
of `&str`.

## Solution

I changed the parameter types to `&[&str]` and `&str` respectively for
consistency with other functions. I also updated all relevant code.

## Type of Change

- [x] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- [ ] This change requires a documentation update
- [ ] Infrastructure change (CI configs, etc)
- [ ] Non-code change (docs, etc)
- [ ] None of the above: (explain here)

## Test Plan

Test cases should pass.

Author: ericapywang

codegen/apis

@@ -293,7 +293,7 @@ impl Index {
     /// The query operation searches a namespace using a query vector. It retrieves the ids of the most similar items in a namespace, along with their similarity scores.
     ///
     /// ### Arguments
-    /// * `id: String` - The id of the query vector.
+    /// * `id: &str` - The id of the query vector.
     /// * `top_k: u32` - The number of results to return.
     /// * `namespace: &Namespace` - The namespace to query. Default is "".
     /// * `filter: Option<Metadata>` - The filter to apply to limit your search by vector metadata.
@@ -317,21 +317,21 @@ impl Index {
     /// let mut index = pinecone.index("index-host").await.unwrap();
     ///
     /// // Query the vector with id "vector-id" in the namespace "namespace"
-    /// let response = index.query_by_id("vector-id".to_string(), 10, &Namespace::default(), None, None, None).await.unwrap();
+    /// let response = index.query_by_id("vector-id", 10, &Namespace::default(), None, None, None).await.unwrap();
     /// # Ok(())
     /// # }
     /// ```
     pub async fn query_by_id(
         &mut self,
-        id: String,
+        id: &str,
         top_k: u32,
         namespace: &Namespace,
         filter: Option<Metadata>,
         include_values: Option<bool>,
         include_metadata: Option<bool>,
     ) -> Result<QueryResponse, PineconeError> {
         let request = pb::QueryRequest {
-            id,
+            id: id.to_string(),
             top_k,
             namespace: namespace.name.clone(),
             filter,
@@ -547,7 +547,7 @@ impl Index {
     /// The fetch operation retrieves vectors by ID from a namespace.
     ///
     /// ### Arguments
-    /// * `ids: &[String]` - The ids of vectors to fetch.
+    /// * `ids: &[&str]` - The ids of vectors to fetch.
     /// * `namespace: &Namespace` - The namespace to fetch vectors from. Default is "".
     ///
     /// ### Return
@@ -567,7 +567,7 @@ impl Index {
     /// // Connect to index host url
     /// let mut index = pinecone.index("index-host").await.unwrap();
     ///
-    /// let vectors = &["1".to_string(), "2".to_string()];
+    /// let vectors = &["1", "2"];
     ///
     /// // Fetch vectors from the default namespace that have the ids in the list
     /// let response = index.fetch(vectors, &Default::default()).await.unwrap();
@@ -576,11 +576,12 @@ impl Index {
     /// ```
     pub async fn fetch(
         &mut self,
-        ids: &[String],
+        ids: &[&str],
         namespace: &Namespace,
     ) -> Result<FetchResponse, PineconeError> {
+        let ids = ids.iter().map(|id| id.to_string()).collect::<Vec<String>>();
         let request = pb::FetchRequest {
-            ids: ids.to_vec(),
+            ids,
             namespace: namespace.name.clone(),
         };
 

pinecone_sdk/src/pinecone/data.rs

@@ -649,7 +649,7 @@ async fn test_query_by_id() -> Result<(), PineconeError> {
         .expect("Failed to target index");
 
     let _query_response = index
-        .query_by_id("1".to_string(), 10, &Namespace::default(), None, None, None)
+        .query_by_id("1", 10, &Namespace::default(), None, None, None)
         .await
         .expect("Failed to query");
 
@@ -983,7 +983,7 @@ async fn test_fetch_vectors() -> Result<(), PineconeError> {
 
     let fetch_response = index
         .fetch(
-            &["1".to_string(), "2".to_string()],
+            &["1", "2"],
             namespace,
         )
         .await
@@ -1034,7 +1034,7 @@ async fn test_fetch_no_match() -> Result<(), PineconeError> {
         .expect("Failed to target index");
 
     let fetch_response = index
-        .fetch(&["invalid-id1".to_string(), "invalid-id2".to_string()], &Default::default())
+        .fetch(&["invalid-id1", "invalid-id2"], &Default::default())
         .await
         .expect("Failed to fetch vectors");