Skip to content

Commit dafea94

Browse files
committed
Clarify add_branch_result diagram
1 parent 6fce213 commit dafea94

File tree

1 file changed

+17
-18
lines changed
  • datafusion/physical-expr/src/expressions

1 file changed

+17
-18
lines changed

datafusion/physical-expr/src/expressions/case.rs

Lines changed: 17 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -391,26 +391,25 @@ impl ResultBuilder {
391391
///
392392
/// The diagram below shows a situation where a when expression matched rows 1 and 4 of the
393393
/// record batch. The then expression produced the value array `[A, D]`.
394-
/// After adding this result, the result array will have been added to `Partial::arrays` and
395-
/// `Partial::indices` will have been updated at indexes 1 and 4.
394+
/// After adding this result, the result array will have been added to `partial arrays` and
395+
/// `partial indices` will have been updated at indexes `1` and `4`.
396396
///
397397
/// ```text
398-
/// ┌─────────┐ ┌─────────┐┌───────────┐ ┌─────────┐┌───────────┐
399-
/// │ C │ │ None ││┌─────────┐│ │ None ││┌─────────┐│
400-
/// ├─────────┤ ├─────────┤││ A ││ ├─────────┤││ A ││
401-
/// │ D │ │ None ││└─────────┘│ │ 2 ││└─────────┘│
402-
/// └─────────┘ ├─────────┤│┌─────────┐│ add_branch_result( ├─────────┤│┌─────────┐│
403-
/// value │ 0 │││ B ││ row indices, │ 0 │││ B ││
404-
/// ├─────────┤│└─────────┘│ value ├─────────┤│└─────────┘│
405-
/// │ None ││ │ ) │ None ││┌─────────┐│
406-
/// ┌─────────┐ ├─────────┤│ │ ─────────────────────────▶ ├─────────┤││ C ││
407-
/// │ 1 │ │ None ││ │ │ 2 ││├─────────┤│
408-
/// ├─────────┤ ├─────────┤│ │ ├─────────┤││ D ││
409-
/// │ 4 │ │ 1 ││ │ │ 1 ││└─────────┘│
410-
/// └─────────┘ └─────────┘└───────────┘ └─────────┘└───────────┘
411-
/// row indices
412-
/// partial partial partial partial
413-
/// indices arrays indices arrays
398+
/// ┌─────────┐ ┌─────────┐┌───────────┐ ┌─────────┐┌───────────┐
399+
/// │ C │ │ 0: None ││┌ 0 ──────┐│ │ 0: None ││┌ 0 ──────┐│
400+
/// ├─────────┤ ├─────────┤││ A ││ ├─────────┤││ A ││
401+
/// │ D │ │ 1: None ││└─────────┘│ │ 1: 2 ││└─────────┘│
402+
/// └─────────┘ ├─────────┤│┌ 1 ──────┐│ add_branch_result( ├─────────┤│┌ 1 ──────┐│
403+
/// matching │ 2: 0 │││ B ││ row indices, │ 2: 0 │││ B ││
404+
/// 'then' values ├─────────┤│└─────────┘│ value ├─────────┤│└─────────┘│
405+
/// │ 3: None ││ │ ) │ 3: None ││┌ 2 ──────┐│
406+
/// ┌─────────┐ ├─────────┤│ │ ─────────────────────────▶ ├─────────┤││ C ││
407+
/// │ 1 │ │ 4: None ││ │ │ 4: 2 ││├─────────┤│
408+
/// ├─────────┤ ├─────────┤│ │ ├─────────┤││ D ││
409+
/// │ 4 │ │ 5: 1 ││ │ │ 5: 1 ││└─────────┘│
410+
/// └─────────┘ └─────────┘└───────────┘ └─────────┘└───────────┘
411+
/// row indices partial partial partial partial
412+
/// indices arrays indices arrays
414413
/// ```
415414
fn add_branch_result(
416415
&mut self,

0 commit comments

Comments
 (0)