Skip to content

Commit

Permalink
Merge pull request scylladb#970 from wprzytula/value_tests
Browse files Browse the repository at this point in the history
Deserialization refactor: introduce new deserialization traits
  • Loading branch information
wprzytula authored Jun 13, 2024
2 parents 3d3c353 + 30d5149 commit ab9a80e
Show file tree
Hide file tree
Showing 10 changed files with 4,460 additions and 225 deletions.
1 change: 1 addition & 0 deletions scylla-cql/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ assert_matches = "1.5.0"
criterion = "0.4" # Note: v0.5 needs at least rust 1.70.0
# Use large-dates feature to test potential edge cases
time = { version = "0.3.21", features = ["large-dates"] }
uuid = { version = "1.0", features = ["v4"] }

[[bench]]
name = "benchmark"
Expand Down
3 changes: 3 additions & 0 deletions scylla-cql/src/frame/frame_errors.rs
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
use super::TryFromPrimitiveError;
use crate::cql_to_rust::CqlTypeError;
use crate::frame::value::SerializeValuesError;
use crate::types::deserialize::DeserializationError;
use crate::types::serialize::SerializationError;
use thiserror::Error;

Expand Down Expand Up @@ -39,6 +40,8 @@ pub enum ParseError {
#[error("Could not deserialize frame: {0}")]
BadIncomingData(String),
#[error(transparent)]
DeserializationError(#[from] DeserializationError),
#[error(transparent)]
IoError(#[from] std::io::Error),
#[error("type not yet implemented, id: {0}")]
TypeNotImplemented(u16),
Expand Down
Loading

0 comments on commit ab9a80e

Please sign in to comment.