forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commita189ed4
committed
Add tests for more row patterns with COPY FROM .. (ON_ERROR ignore)
While digging into the code of this feature, I got confused by the factthat a line is skipped when a value cannot be converted to its expectedattribute even if the line has fewer attributes than the targetrelation. The tests had a check for the case of an empty line, thiscommit a couple more patterns where a line is incomplete, but skippedbecause of a conversion error.Reviewed-by: Bharath RupireddyDiscussion:https://postgr.es/m/Ze_7kZqexdt0BiyC@paquier.xyz1 parent0b84f5c commita189ed4
2 files changed
+6
-2
lines changedLines changed: 3 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
730 | 730 |
| |
731 | 731 |
| |
732 | 732 |
| |
733 |
| - | |
| 733 | + | |
734 | 734 |
| |
735 | 735 |
| |
736 | 736 |
| |
737 | 737 |
| |
738 | 738 |
| |
739 |
| - | |
| 739 | + | |
| 740 | + | |
740 | 741 |
| |
741 | 742 |
| |
742 | 743 |
| |
|
Lines changed: 3 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
515 | 515 |
| |
516 | 516 |
| |
517 | 517 |
| |
| 518 | + | |
| 519 | + | |
| 520 | + | |
518 | 521 |
| |
519 | 522 |
| |
520 | 523 |
| |
|
0 commit comments
Comments
(0)