Skip to content

Commit af65894

Browse files
update comment
1 parent b44291f commit af65894

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

crates/core/src/subscription/delta.rs

+4-5
Original file line numberDiff line numberDiff line change
@@ -30,12 +30,9 @@ pub fn eval_delta<'a, Tx: Datastore + DeltaStore>(
3030
let mut duplicate_rows_evaluated = 0;
3131
let mut duplicate_rows_sent = 0;
3232

33-
// Query plans for joins may return redundant rows,
34-
// but we track row counts to avoid sending them to clients.
35-
//
36-
// Single table plans will never return redundant rows,
37-
// so there's no need to track row counts.
3833
if !plan.is_join() {
34+
// Single table plans will never return redundant rows,
35+
// so there's no need to track row counts.
3936
plan.for_each_insert(tx, metrics, &mut |row| {
4037
inserts.push(row.into());
4138
Ok(())
@@ -46,6 +43,8 @@ pub fn eval_delta<'a, Tx: Datastore + DeltaStore>(
4643
Ok(())
4744
})?;
4845
} else {
46+
// Query plans for joins may return redundant rows.
47+
// We track row counts to avoid sending them to clients.
4948
let mut insert_counts = HashMap::new();
5049
let mut delete_counts = HashMap::new();
5150

0 commit comments

Comments
 (0)