Skip to content

Commit a4b6561

Browse files
committed
refactor: better name
1 parent d8bd11a commit a4b6561

File tree

1 file changed

+17
-17
lines changed

1 file changed

+17
-17
lines changed

datafusion/optimizer/src/propagate_empty_relation.rs

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -449,7 +449,7 @@ mod tests {
449449
assert_together_optimized_plan_eq(plan, expected)
450450
}
451451

452-
fn empty_left_and_right_lp(
452+
fn empty_left_and_right_lp_empty(
453453
left_empty: bool,
454454
right_empty: bool,
455455
join_type: JoinType,
@@ -503,57 +503,57 @@ mod tests {
503503
#[test]
504504
fn test_join_empty_propagation_rules() -> Result<()> {
505505
// test left join with empty left
506-
empty_left_and_right_lp(true, false, JoinType::Left, true)?;
506+
empty_left_and_right_lp_empty(true, false, JoinType::Left, true)?;
507507

508508
// test right join with empty right
509-
empty_left_and_right_lp(false, true, JoinType::Right, true)?;
509+
empty_left_and_right_lp_empty(false, true, JoinType::Right, true)?;
510510

511511
// test left semi join with empty left
512-
empty_left_and_right_lp(true, false, JoinType::LeftSemi, true)?;
512+
empty_left_and_right_lp_empty(true, false, JoinType::LeftSemi, true)?;
513513

514514
// test left semi join with empty right
515-
empty_left_and_right_lp(false, true, JoinType::LeftSemi, true)?;
515+
empty_left_and_right_lp_empty(false, true, JoinType::LeftSemi, true)?;
516516

517517
// test right semi join with empty left
518-
empty_left_and_right_lp(true, false, JoinType::RightSemi, true)?;
518+
empty_left_and_right_lp_empty(true, false, JoinType::RightSemi, true)?;
519519

520520
// test right semi join with empty right
521-
empty_left_and_right_lp(false, true, JoinType::RightSemi, true)?;
521+
empty_left_and_right_lp_empty(false, true, JoinType::RightSemi, true)?;
522522

523523
// test left anti join empty left
524-
empty_left_and_right_lp(true, false, JoinType::LeftAnti, true)?;
524+
empty_left_and_right_lp_empty(true, false, JoinType::LeftAnti, true)?;
525525

526526
// test right anti join empty right
527-
empty_left_and_right_lp(false, true, JoinType::RightAnti, true)
527+
empty_left_and_right_lp_empty(false, true, JoinType::RightAnti, true)
528528
}
529529

530530
#[test]
531531
fn test_join_empty_propagation_rules_noop() -> Result<()> {
532532
// these cases should not result in an empty relation
533533

534534
// test left join with empty right
535-
empty_left_and_right_lp(false, true, JoinType::Left, false)?;
535+
empty_left_and_right_lp_empty(false, true, JoinType::Left, false)?;
536536

537537
// test right join with empty left
538-
empty_left_and_right_lp(true, false, JoinType::Right, false)?;
538+
empty_left_and_right_lp_empty(true, false, JoinType::Right, false)?;
539539

540540
// test left semi with non-empty left and right
541-
empty_left_and_right_lp(false, false, JoinType::LeftSemi, false)?;
541+
empty_left_and_right_lp_empty(false, false, JoinType::LeftSemi, false)?;
542542

543543
// test right semi with non-empty left and right
544-
empty_left_and_right_lp(false, false, JoinType::RightSemi, false)?;
544+
empty_left_and_right_lp_empty(false, false, JoinType::RightSemi, false)?;
545545

546546
// test left anti join with non-empty left and right
547-
empty_left_and_right_lp(false, false, JoinType::LeftAnti, false)?;
547+
empty_left_and_right_lp_empty(false, false, JoinType::LeftAnti, false)?;
548548

549549
// test left anti with non-empty left and empty right
550-
empty_left_and_right_lp(false, true, JoinType::LeftAnti, false)?;
550+
empty_left_and_right_lp_empty(false, true, JoinType::LeftAnti, false)?;
551551

552552
// test right anti join with non-empty left and right
553-
empty_left_and_right_lp(false, false, JoinType::RightAnti, false)?;
553+
empty_left_and_right_lp_empty(false, false, JoinType::RightAnti, false)?;
554554

555555
// test right anti with empty left and non-empty right
556-
empty_left_and_right_lp(true, false, JoinType::RightAnti, false)
556+
empty_left_and_right_lp_empty(true, false, JoinType::RightAnti, false)
557557
}
558558

559559
#[test]

0 commit comments

Comments
 (0)