Skip to content

Commit 9869eae

Browse files
emilkemilk
authored
Include "outer nullability" when we format column datatypes (#11339)
1 parent 6808754 commit 9869eae

File tree

34 files changed

+466
-454
lines changed

34 files changed

+466
-454
lines changed

‎crates/store/re_chunk/tests/snapshots/formatting__format_chunk.snap‎

Lines changed: 18 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -2,23 +2,23 @@
22
source: crates/store/re_chunk/tests/formatting.rs
33
expression: "format!(\"{:240}\", chunk)"
44
---
5-
┌────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┐
6-
│ METADATA: │
7-
│ * entity_path: /this/that │
5+
┌──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┐
6+
│ METADATA: │
7+
│ * entity_path: /this/that │
88
│ * heap_size_bytes: [**REDACTED**] │
9-
│ * id: chunk_0000000000661EFDf2e3b19f7c045f15 │
9+
│ * id: chunk_0000000000661EFDf2e3b19f7c045f15 │
1010
│ * version: [**REDACTED**] │
11-
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┤
12-
│ ┌──────────────────────────────────────────────┬──────────────────────┬───────────────────────────────┬─────────────────────────────────┬────────────────────────────────────┐ │
13-
│ │ RowId ┆ frame_nr ┆ log_time ┆ my_index ┆ example.MyPoints:colors │ │
14-
│ │ --- ┆ --- ┆ --- ┆ --- ┆ --- │ │
15-
│ │ type: FixedSizeBinary[16] ┆ type: i64 ┆ type: Timestamp(ns) ┆ type: List[nullable u64] ┆ type: List[nullable u32] │ │
16-
│ │ ARROW:extension:metadata: ┆ index_name: frame_nr ┆ index_name: log_time ┆ component: my_index ┆ archetype: example.MyPoints │ │
17-
│ │ {"namespace":"row"} ┆ is_sorted: true ┆ is_sorted: true ┆ component_type: example.MyIndex ┆ component: example.MyPoints:colors │ │
18-
│ │ ARROW:extension:name: TUID ┆ kind: index ┆ kind: index ┆ kind: data ┆ component_type: example.MyColor │ │
19-
│ │ is_sorted: true ┆ ┆ ┆ ┆ kind: data │ │
20-
│ │ kind: control ┆ ┆ ┆ ┆ │ │
21-
│ ╞══════════════════════════════════════════════╪══════════════════════╪═══════════════════════════════╪═════════════════════════════════╪════════════════════════════════════╡ │
22-
│ │ row_0000000067816A6Bb4b8c1254d40007b ┆ 1 ┆ 2025-01-10T18:43:42.123456789 ┆ [0, 1, 2] ┆ [0, 1, 2] │ │
23-
│ └──────────────────────────────────────────────┴──────────────────────┴───────────────────────────────┴─────────────────────────────────┴────────────────────────────────────┘ │
24-
└────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘
11+
├╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌╌┤
12+
│ ┌──────────────────────────────────────────────┬──────────────────────┬───────────────────────────────┬───────────────────────────────────┬────────────────────────────────────┐ │
13+
│ │ RowId ┆ frame_nr ┆ log_time ┆ my_index ┆ example.MyPoints:colors │ │
14+
│ │ --- ┆ --- ┆ --- ┆ --- ┆ --- │ │
15+
│ │ type: FixedSizeBinary[16] ┆ type: nullable i64 ┆ type: nullable Timestamp(ns) ┆ type: nullable List[nullable u64] ┆ type: nullable List[nullable u32] │ │
16+
│ │ ARROW:extension:metadata: ┆ index_name: frame_nr ┆ index_name: log_time ┆ component: my_index ┆ archetype: example.MyPoints │ │
17+
│ │ {"namespace":"row"} ┆ is_sorted: true ┆ is_sorted: true ┆ component_type: example.MyIndex ┆ component: example.MyPoints:colors │ │
18+
│ │ ARROW:extension:name: TUID ┆ kind: index ┆ kind: index ┆ kind: data ┆ component_type: example.MyColor │ │
19+
│ │ is_sorted: true ┆ ┆ ┆ ┆ kind: data │ │
20+
│ │ kind: control ┆ ┆ ┆ ┆ │ │
21+
│ ╞══════════════════════════════════════════════╪══════════════════════╪═══════════════════════════════╪═══════════════════════════════════╪════════════════════════════════════╡ │
22+
│ │ row_0000000067816A6Bb4b8c1254d40007b ┆ 1 ┆ 2025-01-10T18:43:42.123456789 ┆ [0, 1, 2] ┆ [0, 1, 2] │ │
23+
│ └──────────────────────────────────────────────┴──────────────────────┴───────────────────────────────┴───────────────────────────────────┴────────────────────────────────────┘ │
24+
└──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘

‎crates/store/re_chunk/tests/snapshots/formatting__format_chunk_redacted.snap‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ expression: "format!(\"{:240}\", ChunkRedacted(chunk))"
1212
│ ┌──────────────────────────────────────────────┬────────────────────────────┬───────────────────────────────┬───────────────────────────────────────┬──────────────────────────────────────────┐ │
1313
│ │ rerun.controls.RowId ┆ frame_nr ┆ log_time ┆ my_index ┆ example.MyPoints:colors │ │
1414
│ │ --- ┆ --- ┆ --- ┆ --- ┆ --- │ │
15-
│ │ type: FixedSizeBinary[16] ┆ type: i64 ┆ type: Timestamp(ns) ┆ type: List[nullable u64] ┆ type: List[nullable u32] │ │
15+
│ │ type: FixedSizeBinary[16] ┆ type: nullable i64 ┆ type: nullable Timestamp(ns) ┆ type: nullable List[nullable u64] ┆ type: nullable List[nullable u32] │ │
1616
│ │ ARROW:extension:metadata: ┆ rerun:index_name: frame_nr ┆ rerun:index_name: log_time ┆ rerun:component: my_index ┆ rerun:archetype: example.MyPoints │ │
1717
│ │ {"namespace":"row"} ┆ rerun:is_sorted: true ┆ rerun:is_sorted: true ┆ rerun:component_type: example.MyIndex ┆ rerun:component: example.MyPoints:colors │ │
1818
│ │ ARROW:extension:name: rerun.datatypes.TUID ┆ rerun:kind: index ┆ rerun:kind: index ┆ rerun:kind: data ┆ rerun:component_type: example.MyColor │ │

0 commit comments

Comments
 (0)