Skip to content

Commit 4c852e2

Browse files
authored
refactor: remove innecessary serde (#15040)
* refactor: remove innecessary serde * refactor: remove innecessary serde
1 parent ac27ee7 commit 4c852e2

File tree

7 files changed

+59
-59
lines changed

7 files changed

+59
-59
lines changed

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

Lines changed: 38 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ impl Display for TableIdent {
6666
}
6767
}
6868

69-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, Eq, PartialEq, Default)]
69+
#[derive(Clone, Debug, Eq, PartialEq, Default)]
7070
pub struct TableNameIdent {
7171
pub tenant: String,
7272
pub db_name: String,
@@ -112,7 +112,7 @@ impl Display for TableNameIdent {
112112
}
113113
}
114114

115-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, Eq, PartialEq, Default)]
115+
#[derive(Clone, Debug, Eq, PartialEq, Default)]
116116
pub struct DBIdTableName {
117117
pub db_id: u64,
118118
pub table_name: String,
@@ -124,7 +124,7 @@ impl Display for DBIdTableName {
124124
}
125125
}
126126

127-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, Eq, PartialEq, Default)]
127+
#[derive(Clone, Debug, Eq, PartialEq, Default)]
128128
pub struct TableId {
129129
pub table_id: u64,
130130
}
@@ -141,7 +141,7 @@ impl Display for TableId {
141141
}
142142
}
143143

144-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, Eq, PartialEq, Default)]
144+
#[derive(Clone, Debug, Eq, PartialEq, Default)]
145145
pub struct TableIdListKey {
146146
pub db_id: u64,
147147
pub table_name: String,
@@ -458,7 +458,7 @@ impl Display for TableIdList {
458458
}
459459
}
460460

461-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
461+
#[derive(Clone, Debug, PartialEq, Eq)]
462462
pub struct CreateTableReq {
463463
pub create_option: CreateOption,
464464
pub name_ident: TableNameIdent,
@@ -554,7 +554,7 @@ pub struct DropTableReply {
554554
pub spec_vec: Option<(Vec<ShareSpec>, Vec<ShareTableInfoMap>)>,
555555
}
556556

557-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
557+
#[derive(Clone, Debug, PartialEq, Eq)]
558558
pub struct UndropTableReq {
559559
pub name_ident: TableNameIdent,
560560
}
@@ -583,10 +583,10 @@ impl Display for UndropTableReq {
583583
}
584584
}
585585

586-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
586+
#[derive(Clone, Debug, PartialEq, Eq)]
587587
pub struct UndropTableReply {}
588588

589-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
589+
#[derive(Clone, Debug, PartialEq, Eq)]
590590
pub struct RenameTableReq {
591591
pub if_exists: bool,
592592
pub name_ident: TableNameIdent,
@@ -620,12 +620,12 @@ impl Display for RenameTableReq {
620620
}
621621
}
622622

623-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
623+
#[derive(Clone, Debug, PartialEq, Eq)]
624624
pub struct RenameTableReply {
625625
pub table_id: u64,
626626
}
627627

628-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
628+
#[derive(Clone, Debug, PartialEq, Eq)]
629629
pub struct UpsertTableOptionReq {
630630
pub table_id: u64,
631631
pub seq: MatchSeq,
@@ -644,7 +644,7 @@ pub struct UpdateStreamMetaReq {
644644
pub options: BTreeMap<String, String>,
645645
}
646646

647-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
647+
#[derive(Clone, Debug, PartialEq, Eq)]
648648
pub struct UpdateTableMetaReq {
649649
pub table_id: u64,
650650
pub seq: MatchSeq,
@@ -685,15 +685,15 @@ impl Display for UpsertTableOptionReq {
685685
}
686686
}
687687

688-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
688+
#[derive(Clone, Debug, PartialEq, Eq)]
689689
pub enum SetTableColumnMaskPolicyAction {
690690
// new mask name, old mask name(if any)
691691
Set(String, Option<String>),
692692
// prev mask name
693693
Unset(String),
694694
}
695695

696-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
696+
#[derive(Clone, Debug, PartialEq, Eq)]
697697
pub struct SetTableColumnMaskPolicyReq {
698698
pub tenant: String,
699699
pub table_id: u64,
@@ -702,17 +702,17 @@ pub struct SetTableColumnMaskPolicyReq {
702702
pub action: SetTableColumnMaskPolicyAction,
703703
}
704704

705-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
705+
#[derive(Clone, Debug, PartialEq, Eq)]
706706
pub struct SetTableColumnMaskPolicyReply {
707707
pub share_table_info: Option<Vec<ShareTableInfoMap>>,
708708
}
709709

710-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
710+
#[derive(Clone, Debug, PartialEq, Eq)]
711711
pub struct UpsertTableOptionReply {
712712
pub share_table_info: Option<Vec<ShareTableInfoMap>>,
713713
}
714714

715-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
715+
#[derive(Clone, Debug, PartialEq, Eq)]
716716
pub struct UpdateTableMetaReply {
717717
pub share_table_info: Option<Vec<ShareTableInfoMap>>,
718718
}
@@ -745,7 +745,7 @@ impl Display for CreateTableIndexReq {
745745
}
746746
}
747747

748-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, Eq, PartialEq)]
748+
#[derive(Clone, Debug, Eq, PartialEq)]
749749
pub struct CreateTableIndexReply {}
750750

751751
#[derive(Clone, Debug, PartialEq, Eq)]
@@ -765,10 +765,10 @@ impl Display for DropTableIndexReq {
765765
}
766766
}
767767

768-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
768+
#[derive(Clone, Debug, PartialEq, Eq)]
769769
pub struct DropTableIndexReply {}
770770

771-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
771+
#[derive(Clone, Debug, PartialEq, Eq)]
772772
pub struct GetTableReq {
773773
pub inner: TableNameIdent,
774774
}
@@ -799,7 +799,7 @@ impl GetTableReq {
799799
}
800800
}
801801

802-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
802+
#[derive(Clone, Debug, PartialEq, Eq)]
803803
pub struct ListTableReq {
804804
pub inner: DatabaseNameIdent,
805805
}
@@ -823,7 +823,7 @@ impl ListTableReq {
823823
}
824824
}
825825

826-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
826+
#[derive(Clone, Debug, PartialEq, Eq)]
827827
pub enum TableInfoFilter {
828828
// if datatime is some, filter only dropped tables which drop time before that,
829829
// else filter all dropped tables
@@ -838,37 +838,37 @@ pub enum TableInfoFilter {
838838
All,
839839
}
840840

841-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
841+
#[derive(Clone, Debug, PartialEq, Eq)]
842842
pub struct ListDroppedTableReq {
843843
pub inner: DatabaseNameIdent,
844844
pub filter: TableInfoFilter,
845845
pub limit: Option<usize>,
846846
}
847847

848-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
848+
#[derive(Clone, Debug, PartialEq, Eq)]
849849
pub enum DroppedId {
850850
// db id, db name
851851
Db(u64, String),
852852
// db id, table id, table name
853853
Table(u64, u64, String),
854854
}
855855

856-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
856+
#[derive(Clone, Debug, PartialEq, Eq)]
857857
pub struct ListDroppedTableResp {
858858
pub drop_table_infos: Vec<Arc<TableInfo>>,
859859
pub drop_ids: Vec<DroppedId>,
860860
}
861861

862-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
862+
#[derive(Clone, Debug, PartialEq, Eq)]
863863
pub struct GcDroppedTableReq {
864864
pub tenant: String,
865865
pub drop_ids: Vec<DroppedId>,
866866
}
867867

868-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
868+
#[derive(Clone, Debug, PartialEq, Eq)]
869869
pub struct GcDroppedTableResp {}
870870

871-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, Eq, PartialEq, Default)]
871+
#[derive(Clone, Debug, Eq, PartialEq, Default)]
872872
pub struct CountTablesKey {
873873
pub tenant: String,
874874
}
@@ -888,7 +888,7 @@ impl CountTablesKey {
888888
}
889889

890890
/// count tables for a tenant
891-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
891+
#[derive(Clone, Debug, PartialEq, Eq)]
892892
pub struct CountTablesReq {
893893
pub tenant: String,
894894
}
@@ -898,7 +898,7 @@ pub struct CountTablesReply {
898898
pub count: u64,
899899
}
900900

901-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, Default, Eq, PartialEq)]
901+
#[derive(Clone, Debug, Default, Eq, PartialEq)]
902902
pub struct TableIdToName {
903903
pub table_id: u64,
904904
}
@@ -909,41 +909,41 @@ impl Display for TableIdToName {
909909
}
910910
}
911911

912-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, Eq, PartialEq, Default)]
912+
#[derive(Clone, Debug, Eq, PartialEq, Default)]
913913
pub struct TableCopiedFileNameIdent {
914914
pub table_id: u64,
915915
pub file: String,
916916
}
917917

918-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, Eq, PartialEq, Default)]
918+
#[derive(Clone, Debug, Eq, PartialEq, Default)]
919919
pub struct TableCopiedFileInfo {
920920
pub etag: Option<String>,
921921
pub content_length: u64,
922922
pub last_modified: Option<DateTime<Utc>>,
923923
}
924924

925-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
925+
#[derive(Clone, Debug, PartialEq, Eq)]
926926
pub struct GetTableCopiedFileReq {
927927
pub table_id: u64,
928928
pub files: Vec<String>,
929929
}
930930

931-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
931+
#[derive(Clone, Debug, PartialEq, Eq)]
932932
pub struct GetTableCopiedFileReply {
933933
pub file_info: BTreeMap<String, TableCopiedFileInfo>,
934934
}
935935

936-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
936+
#[derive(Clone, Debug, PartialEq, Eq)]
937937
pub struct UpsertTableCopiedFileReq {
938938
pub file_info: BTreeMap<String, TableCopiedFileInfo>,
939939
pub expire_at: Option<u64>,
940940
pub fail_if_duplicated: bool,
941941
}
942942

943-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
943+
#[derive(Clone, Debug, PartialEq, Eq)]
944944
pub struct UpsertTableCopiedFileReply {}
945945

946-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
946+
#[derive(Clone, Debug, PartialEq, Eq)]
947947
pub struct TruncateTableReq {
948948
pub table_id: u64,
949949
/// Specify the max number copied file to delete in every sub-transaction.
@@ -952,10 +952,10 @@ pub struct TruncateTableReq {
952952
pub batch_size: Option<u64>,
953953
}
954954

955-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
955+
#[derive(Clone, Debug, PartialEq, Eq)]
956956
pub struct TruncateTableReply {}
957957

958-
#[derive(serde::Serialize, serde::Deserialize, Clone, Debug, PartialEq, Eq)]
958+
#[derive(Clone, Debug, PartialEq, Eq)]
959959
pub struct EmptyProto {}
960960

961961
mod kvapi_key_impl {

0 commit comments

Comments
 (0)