Commit c2dd021
[SPARK-52632][SQL] Pretty display V2 write plan nodes
### What changes were proposed in this pull request?
Pretty display V2 write plan nodes by overriding `def stringArgs`.
### Why are the changes needed?
Better UX for UI display and `EXPLAIN` output
### Does this PR introduce _any_ user-facing change?
Yes, change affects UI display and `EXPLAIN` output for V2 write cases.
### How was this patch tested?
Use AppendData as an example.
Before
<img width="1721" alt="Xnip2025-07-01_16-39-59" src="https://github.com/user-attachments/assets/9dcab5e2-c768-45a3-8c0b-f7059c8c1389" />
After
<img width="1712" alt="Xnip2025-07-01_16-30-00" src="https://github.com/user-attachments/assets/fd3fdd7c-da5e-49be-b801-f3debde02808" />
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #51332 from pan3793/SPARK-52632.
Authored-by: Cheng Pan <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>1 parent 3fb4adb commit c2dd021
File tree
2 files changed
+9
-4
lines changed- sql/core/src/main/scala/org/apache/spark/sql/execution/datasources
- noop
- v2
2 files changed
+9
-4
lines changedLines changed: 5 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
27 | 27 | | |
28 | 28 | | |
29 | 29 | | |
| 30 | + | |
30 | 31 | | |
31 | 32 | | |
32 | 33 | | |
| |||
48 | 49 | | |
49 | 50 | | |
50 | 51 | | |
| 52 | + | |
51 | 53 | | |
52 | 54 | | |
53 | 55 | | |
| |||
59 | 61 | | |
60 | 62 | | |
61 | 63 | | |
| 64 | + | |
62 | 65 | | |
63 | 66 | | |
64 | 67 | | |
| |||
67 | 70 | | |
68 | 71 | | |
69 | 72 | | |
| 73 | + | |
70 | 74 | | |
71 | 75 | | |
72 | 76 | | |
| |||
86 | 90 | | |
87 | 91 | | |
88 | 92 | | |
| 93 | + | |
89 | 94 | | |
90 | 95 | | |
91 | 96 | | |
| |||
Lines changed: 4 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
303 | 303 | | |
304 | 304 | | |
305 | 305 | | |
306 | | - | |
307 | | - | |
308 | 306 | | |
309 | 307 | | |
310 | 308 | | |
| |||
328 | 326 | | |
329 | 327 | | |
330 | 328 | | |
331 | | - | |
332 | | - | |
333 | 329 | | |
334 | 330 | | |
335 | 331 | | |
| |||
349 | 345 | | |
350 | 346 | | |
351 | 347 | | |
| 348 | + | |
| 349 | + | |
352 | 350 | | |
353 | 351 | | |
354 | 352 | | |
| |||
367 | 365 | | |
368 | 366 | | |
369 | 367 | | |
| 368 | + | |
| 369 | + | |
370 | 370 | | |
371 | 371 | | |
372 | 372 | | |
| |||
0 commit comments