Skip to content

Commit c3871fb

Browse files
authored
Derive columnar for log events (#557)
Signed-off-by: Moritz Hoffmann <[email protected]>
1 parent a21bf64 commit c3871fb

File tree

1 file changed

+8
-7
lines changed

1 file changed

+8
-7
lines changed

src/logging.rs

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
//! Loggers and logging events for differential dataflow.
22
3+
use columnar::Columnar;
34
use serde::{Deserialize, Serialize};
45

56
/// Logger for differential dataflow events.
@@ -19,7 +20,7 @@ where
1920
}
2021

2122
/// Possible different differential events.
22-
#[derive(Debug, Clone, Ord, PartialOrd, Eq, PartialEq, Serialize, Deserialize)]
23+
#[derive(Debug, Clone, Ord, PartialOrd, Eq, PartialEq, Serialize, Deserialize, Columnar)]
2324
pub enum DifferentialEvent {
2425
/// Batch creation.
2526
Batch(BatchEvent),
@@ -36,7 +37,7 @@ pub enum DifferentialEvent {
3637
}
3738

3839
/// Either the start or end of a merge event.
39-
#[derive(Debug, Clone, Ord, PartialOrd, Eq, PartialEq, Serialize, Deserialize)]
40+
#[derive(Debug, Clone, Ord, PartialOrd, Eq, PartialEq, Serialize, Deserialize, Columnar)]
4041
pub struct BatchEvent {
4142
/// Operator identifier.
4243
pub operator: usize,
@@ -48,7 +49,7 @@ impl From<BatchEvent> for DifferentialEvent { fn from(e: BatchEvent) -> Self { D
4849

4950

5051
/// Either the start or end of a merge event.
51-
#[derive(Debug, Clone, Ord, PartialOrd, Eq, PartialEq, Serialize, Deserialize)]
52+
#[derive(Debug, Clone, Ord, PartialOrd, Eq, PartialEq, Serialize, Deserialize, Columnar)]
5253
pub struct BatcherEvent {
5354
/// Operator identifier.
5455
pub operator: usize,
@@ -65,7 +66,7 @@ pub struct BatcherEvent {
6566
impl From<BatcherEvent> for DifferentialEvent { fn from(e: BatcherEvent) -> Self { DifferentialEvent::Batcher(e) } }
6667

6768
/// Either the start or end of a merge event.
68-
#[derive(Debug, Clone, Ord, PartialOrd, Eq, PartialEq, Serialize, Deserialize)]
69+
#[derive(Debug, Clone, Ord, PartialOrd, Eq, PartialEq, Serialize, Deserialize, Columnar)]
6970
pub struct DropEvent {
7071
/// Operator identifier.
7172
pub operator: usize,
@@ -76,7 +77,7 @@ pub struct DropEvent {
7677
impl From<DropEvent> for DifferentialEvent { fn from(e: DropEvent) -> Self { DifferentialEvent::Drop(e) } }
7778

7879
/// Either the start or end of a merge event.
79-
#[derive(Debug, Clone, Ord, PartialOrd, Eq, PartialEq, Serialize, Deserialize)]
80+
#[derive(Debug, Clone, Ord, PartialOrd, Eq, PartialEq, Serialize, Deserialize, Columnar)]
8081
pub struct MergeEvent {
8182
/// Operator identifier.
8283
pub operator: usize,
@@ -93,7 +94,7 @@ pub struct MergeEvent {
9394
impl From<MergeEvent> for DifferentialEvent { fn from(e: MergeEvent) -> Self { DifferentialEvent::Merge(e) } }
9495

9596
/// A merge failed to complete in time.
96-
#[derive(Debug, Clone, Ord, PartialOrd, Eq, PartialEq, Serialize, Deserialize)]
97+
#[derive(Debug, Clone, Ord, PartialOrd, Eq, PartialEq, Serialize, Deserialize, Columnar)]
9798
pub struct MergeShortfall {
9899
/// Operator identifier.
99100
pub operator: usize,
@@ -106,7 +107,7 @@ pub struct MergeShortfall {
106107
impl From<MergeShortfall> for DifferentialEvent { fn from(e: MergeShortfall) -> Self { DifferentialEvent::MergeShortfall(e) } }
107108

108109
/// Either the start or end of a merge event.
109-
#[derive(Debug, Clone, Ord, PartialOrd, Eq, PartialEq, Serialize, Deserialize)]
110+
#[derive(Debug, Clone, Ord, PartialOrd, Eq, PartialEq, Serialize, Deserialize, Columnar)]
110111
pub struct TraceShare {
111112
/// Operator identifier.
112113
pub operator: usize,

0 commit comments

Comments
 (0)