dynein icon indicating copy to clipboard operation
dynein copied to clipboard

Remove unnecessary `vec!` from `create_temporary_table_with_items` invocations

Open StoneDot opened this issue 9 months ago • 0 comments

Currently we typically use create_temporary_table_with_items with vec!. However, current implementation does not need to use vec! because of the improvement of #218. We can simplify the implementation of caller sides.

    pub async fn create_temporary_table_with_items<ItemIter>(
        &mut self,
        pk: &'static str,
        sk: Option<&'static str>,
        items: ItemIter,
    ) -> Result<String, Box<dyn std::error::Error>>
    where
        ItemIter: IntoIterator<Item = TemporaryItem>,

StoneDot avatar May 17 '24 02:05 StoneDot