Skip to content

Commit 570d808

Browse files
committed
replace custom enum with Result
Signed-off-by: Petros Angelatos <[email protected]>
1 parent e72667e commit 570d808

File tree

1 file changed

+4
-14
lines changed

1 file changed

+4
-14
lines changed

timely/src/dataflow/operators/core/partition.rs

Lines changed: 4 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -58,11 +58,6 @@ impl<G: Scope, C: Container + Data> Partition<G, C> for StreamCore<G, C> {
5858
builder.build(move |_| {
5959
let mut todo = vec![];
6060
move |_frontiers| {
61-
enum SessionState<H, S> {
62-
Handle(H),
63-
Session(S),
64-
}
65-
6661
let mut handles = outputs.iter_mut().map(|o| o.activate()).collect::<Vec<_>>();
6762

6863
// The capability associated with each session in `sessions`.
@@ -76,23 +71,18 @@ impl<G: Scope, C: Container + Data> Partition<G, C> for StreamCore<G, C> {
7671

7772
for (cap, mut data) in todo.drain(..) {
7873
if sessions_cap.as_ref().map_or(true, |s_cap| s_cap.time() != cap.time()) {
79-
sessions = handles
80-
.iter_mut()
81-
.map(|h| Some(SessionState::Handle(h)))
82-
.collect();
74+
sessions = handles.iter_mut().map(|h| Some(Err(h))).collect();
8375
sessions_cap = Some(cap);
8476
}
8577
for datum in data.drain() {
8678
let (part, datum2) = route(datum);
8779

8880
let mut session = match sessions[part as usize].take().unwrap() {
89-
SessionState::Session(s) => s,
90-
SessionState::Handle(handle) => {
91-
handle.session_with_builder(sessions_cap.as_ref().unwrap())
92-
}
81+
Ok(s) => s,
82+
Err(handle) => handle.session_with_builder(sessions_cap.as_ref().unwrap()),
9383
};
9484
session.give(datum2);
95-
sessions[part as usize] = Some(SessionState::Session(session));
85+
sessions[part as usize] = Some(Ok(session));
9686
}
9787
}
9888
}

0 commit comments

Comments
 (0)