Commit f40bf4d
[SPARK-51386][CORE][SQL] Assign name to error conditions _LEGACY_ERROR_TEMP_3300-3302
### What changes were proposed in this pull request?
Assign name to error conditions `_LEGACY_ERROR_TEMP_3300-3302` with sqlState `82001-82003`, all of them are applicable for `SparkOutOfMemoryError`
### Why are the changes needed?
Improve the error framework.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Pass GHA.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #50149 from pan3793/SPARK-51386.
Authored-by: Cheng Pan <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>1 parent 50e00b7 commit f40bf4d
File tree
6 files changed
+42
-22
lines changed- common/utils/src/main/resources/error
- core/src/main/java/org/apache/spark
- memory
- util/collection/unsafe/sort
- sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate
6 files changed
+42
-22
lines changedLines changed: 18 additions & 15 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
14 | 20 | | |
15 | 21 | | |
16 | 22 | | |
| |||
4392 | 4398 | | |
4393 | 4399 | | |
4394 | 4400 | | |
| 4401 | + | |
| 4402 | + | |
| 4403 | + | |
| 4404 | + | |
| 4405 | + | |
| 4406 | + | |
4395 | 4407 | | |
4396 | 4408 | | |
4397 | 4409 | | |
| |||
4645 | 4657 | | |
4646 | 4658 | | |
4647 | 4659 | | |
| 4660 | + | |
| 4661 | + | |
| 4662 | + | |
| 4663 | + | |
| 4664 | + | |
| 4665 | + | |
4648 | 4666 | | |
4649 | 4667 | | |
4650 | 4668 | | |
| |||
9327 | 9345 | | |
9328 | 9346 | | |
9329 | 9347 | | |
9330 | | - | |
9331 | | - | |
9332 | | - | |
9333 | | - | |
9334 | | - | |
9335 | | - | |
9336 | | - | |
9337 | | - | |
9338 | | - | |
9339 | | - | |
9340 | | - | |
9341 | | - | |
9342 | | - | |
9343 | | - | |
9344 | | - | |
9345 | 9348 | | |
9346 | 9349 | | |
9347 | 9350 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6504 | 6504 | | |
6505 | 6505 | | |
6506 | 6506 | | |
6507 | | - | |
| 6507 | + | |
| 6508 | + | |
| 6509 | + | |
| 6510 | + | |
| 6511 | + | |
| 6512 | + | |
| 6513 | + | |
| 6514 | + | |
| 6515 | + | |
| 6516 | + | |
| 6517 | + | |
| 6518 | + | |
| 6519 | + | |
| 6520 | + | |
| 6521 | + | |
| 6522 | + | |
| 6523 | + | |
| 6524 | + | |
6508 | 6525 | | |
6509 | 6526 | | |
6510 | 6527 | | |
| |||
Lines changed: 3 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
278 | 278 | | |
279 | 279 | | |
280 | 280 | | |
281 | | - | |
| 281 | + | |
282 | 282 | | |
283 | 283 | | |
284 | 284 | | |
285 | | - | |
| 285 | + | |
286 | 286 | | |
287 | 287 | | |
288 | 288 | | |
289 | | - | |
| 289 | + | |
290 | 290 | | |
291 | 291 | | |
292 | 292 | | |
| |||
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
216 | 216 | | |
217 | 217 | | |
218 | 218 | | |
219 | | - | |
| 219 | + | |
220 | 220 | | |
221 | 221 | | |
222 | 222 | | |
| |||
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
682 | 682 | | |
683 | 683 | | |
684 | 684 | | |
685 | | - | |
| 685 | + | |
686 | 686 | | |
687 | 687 | | |
688 | 688 | | |
| |||
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregationIterator.scala
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
212 | 212 | | |
213 | 213 | | |
214 | 214 | | |
215 | | - | |
| 215 | + | |
216 | 216 | | |
217 | 217 | | |
218 | 218 | | |
| |||
0 commit comments