Commit b44389c
[SPARK-52618][SQL] Casting TIME(n) to TIME(m)
### What changes were proposed in this pull request?
In the PR, I propose to support casting from TIME(n) to TIME(m) for any n, m in the range [0, 6].
### Why are the changes needed?
Users can change precision of TIME values.
### Does this PR introduce _any_ user-facing change?
No, the TIME data type hasn't been released yet.
### How was this patch tested?
By running new tests:
```
$ build/sbt "sql/testOnly org.apache.spark.sql.SQLQueryTestSuite -- -z time.sql"
```
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #51361 from MaxGekk/cast-time-to-time-2.
Authored-by: Max Gekk <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>1 parent 80ef244 commit b44389c
File tree
5 files changed
+88
-4
lines changed- sql
- catalyst/src
- main/scala/org/apache/spark/sql/catalyst/expressions
- test/scala/org/apache/spark/sql/catalyst/expressions
- core/src/test/resources/sql-tests
- analyzer-results
- inputs
- results
5 files changed
+88
-4
lines changedLines changed: 15 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
134 | 134 | | |
135 | 135 | | |
136 | 136 | | |
| 137 | + | |
| 138 | + | |
137 | 139 | | |
138 | 140 | | |
139 | 141 | | |
| |||
251 | 253 | | |
252 | 254 | | |
253 | 255 | | |
| 256 | + | |
| 257 | + | |
254 | 258 | | |
255 | 259 | | |
256 | 260 | | |
| |||
733 | 737 | | |
734 | 738 | | |
735 | 739 | | |
736 | | - | |
| 740 | + | |
737 | 741 | | |
738 | 742 | | |
739 | 743 | | |
740 | 744 | | |
741 | 745 | | |
742 | 746 | | |
| 747 | + | |
| 748 | + | |
743 | 749 | | |
744 | 750 | | |
745 | 751 | | |
| |||
1149 | 1155 | | |
1150 | 1156 | | |
1151 | 1157 | | |
1152 | | - | |
| 1158 | + | |
1153 | 1159 | | |
1154 | 1160 | | |
1155 | 1161 | | |
| |||
1257 | 1263 | | |
1258 | 1264 | | |
1259 | 1265 | | |
1260 | | - | |
| 1266 | + | |
1261 | 1267 | | |
1262 | 1268 | | |
1263 | 1269 | | |
| |||
1354 | 1360 | | |
1355 | 1361 | | |
1356 | 1362 | | |
| 1363 | + | |
1357 | 1364 | | |
1358 | 1365 | | |
1359 | 1366 | | |
| |||
1374 | 1381 | | |
1375 | 1382 | | |
1376 | 1383 | | |
1377 | | - | |
| 1384 | + | |
| 1385 | + | |
| 1386 | + | |
| 1387 | + | |
| 1388 | + | |
1378 | 1389 | | |
1379 | 1390 | | |
1380 | 1391 | | |
| |||
Lines changed: 23 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1484 | 1484 | | |
1485 | 1485 | | |
1486 | 1486 | | |
| 1487 | + | |
| 1488 | + | |
| 1489 | + | |
| 1490 | + | |
| 1491 | + | |
| 1492 | + | |
| 1493 | + | |
| 1494 | + | |
| 1495 | + | |
| 1496 | + | |
| 1497 | + | |
| 1498 | + | |
| 1499 | + | |
| 1500 | + | |
| 1501 | + | |
| 1502 | + | |
| 1503 | + | |
| 1504 | + | |
| 1505 | + | |
| 1506 | + | |
| 1507 | + | |
| 1508 | + | |
| 1509 | + | |
1487 | 1510 | | |
Lines changed: 21 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
399 | 399 | | |
400 | 400 | | |
401 | 401 | | |
| 402 | + | |
| 403 | + | |
| 404 | + | |
| 405 | + | |
| 406 | + | |
| 407 | + | |
| 408 | + | |
| 409 | + | |
| 410 | + | |
| 411 | + | |
| 412 | + | |
| 413 | + | |
| 414 | + | |
| 415 | + | |
| 416 | + | |
| 417 | + | |
| 418 | + | |
| 419 | + | |
| 420 | + | |
| 421 | + | |
| 422 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
67 | 67 | | |
68 | 68 | | |
69 | 69 | | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
Lines changed: 24 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
498 | 498 | | |
499 | 499 | | |
500 | 500 | | |
| 501 | + | |
| 502 | + | |
| 503 | + | |
| 504 | + | |
| 505 | + | |
| 506 | + | |
| 507 | + | |
| 508 | + | |
| 509 | + | |
| 510 | + | |
| 511 | + | |
| 512 | + | |
| 513 | + | |
| 514 | + | |
| 515 | + | |
| 516 | + | |
| 517 | + | |
| 518 | + | |
| 519 | + | |
| 520 | + | |
| 521 | + | |
| 522 | + | |
| 523 | + | |
| 524 | + | |
0 commit comments