Skip to content

Commit

Permalink
serialize/row: remove SerializedValues::to_old_serialized_values
Browse files Browse the repository at this point in the history
Migration of batches to the new API is complete, so this method is no
longer needed and can finally be removed.
  • Loading branch information
piodul committed Dec 15, 2023
1 parent 2ea431c commit fbeafd7
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions scylla-cql/src/types/serialize/row.rs
Original file line number Diff line number Diff line change
Expand Up @@ -794,14 +794,6 @@ impl SerializedValues {
element_count: values_num,
})
}

/// Temporary function, to be removed when we implement new batching API (right now it is needed in frame::request::mod.rs tests)
// TODO: Remove
pub fn to_old_serialized_values(&self) -> LegacySerializedValues {
let mut frame = Vec::new();
self.write_to_request(&mut frame);
LegacySerializedValues::new_from_frame(&mut frame.as_slice(), false).unwrap()
}
}

impl Default for SerializedValues {
Expand Down

0 comments on commit fbeafd7

Please sign in to comment.