Skip to content

Commit 9334956

Browse files
authored
refactor: remove innecessary serde (#15037)
1 parent 444e052 commit 9334956

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
lines changed

src/meta/app/src/schema/database.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ impl DatabaseNameIdent {
4747
}
4848
}
4949

50-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, Default, Eq, PartialEq)]
50+
#[derive(Clone, Debug, Default, Eq, PartialEq)]
5151
pub struct DatabaseInfo {
5252
pub ident: DatabaseIdent,
5353
pub name_ident: DatabaseNameIdent,
@@ -114,7 +114,7 @@ impl Display for DbIdListKey {
114114
}
115115
}
116116

117-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, Eq, PartialEq)]
117+
#[derive(Clone, Debug, Eq, PartialEq)]
118118
pub struct DatabaseMeta {
119119
pub engine: String,
120120
pub engine_options: BTreeMap<String, String>,
@@ -204,7 +204,7 @@ impl Display for DbIdList {
204204
}
205205
}
206206

207-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
207+
#[derive(Clone, Debug, PartialEq, Eq)]
208208
pub struct CreateDatabaseReq {
209209
pub create_option: CreateOption,
210210
pub name_ident: DatabaseNameIdent,

src/meta/app/src/schema/table.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -508,7 +508,7 @@ impl Display for CreateTableReq {
508508
}
509509
}
510510

511-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, Eq, PartialEq)]
511+
#[derive(Clone, Debug, Eq, PartialEq)]
512512
pub struct CreateTableReply {
513513
pub table_id: u64,
514514
pub new_table: bool,
@@ -519,7 +519,7 @@ pub struct CreateTableReply {
519519
///
520520
/// Dropping a table requires just `table_id`, but when dropping a table, it also needs to update
521521
/// the count of tables belonging to a tenant, which require tenant information.
522-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
522+
#[derive(Clone, Debug, PartialEq, Eq)]
523523
pub struct DropTableByIdReq {
524524
pub if_exists: bool,
525525

@@ -549,7 +549,7 @@ impl Display for DropTableByIdReq {
549549
}
550550
}
551551

552-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
552+
#[derive(Clone, Debug, PartialEq, Eq)]
553553
pub struct DropTableReply {
554554
pub spec_vec: Option<(Vec<ShareSpec>, Vec<ShareTableInfoMap>)>,
555555
}

src/meta/app/src/share/share.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,7 @@ impl Display for ShareGrantObjectName {
175175
}
176176
}
177177

178-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
178+
#[derive(Clone, Debug, PartialEq, Eq)]
179179
pub enum ShareGrantObjectSeqAndId {
180180
// db_meta_seq, db_id, DatabaseMeta
181181
Database(u64, u64, DatabaseMeta),
@@ -716,7 +716,7 @@ pub struct ShareIdent {
716716
pub seq: u64,
717717
}
718718

719-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, Default, Eq, PartialEq)]
719+
#[derive(Clone, Debug, Default, Eq, PartialEq)]
720720
pub struct ShareInfo {
721721
pub ident: ShareIdent,
722722
pub name_ident: ShareNameIdent,

0 commit comments

Comments
 (0)