- Notifications
You must be signed in to change notification settings - Fork28
Commit3a38298
committed
Allow plperl_sv_to_datum to look through scalar refs.
There seems little reason for the policy of throwing error if wefind a ref to something other than a hash or array. Recursivelylook through the ref, instead. This makes the behavior in non-transformcases comparable to what was already instantiated for jsonb_plperl.Note that because we invoke any available transform function beforeconsidering the ref case, it's up to each transform function whetherit wants to play along with this behavior or do something different.Because the previous behavior was just to throw a useless error,this seems unlikely to create any compatibility issues. Still, giventhe lack of field complaints so far, seems best not to back-patch.Discussion:https://postgr.es/m/28336.1528393969@sss.pgh.pa.us1 parente4300a3 commit3a38298
3 files changed
+14
-9
lines changedLines changed: 6 additions & 3 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
763 | 763 |
| |
764 | 764 |
| |
765 | 765 |
| |
766 |
| - | |
| 766 | + | |
767 | 767 |
| |
768 | 768 |
| |
769 | 769 |
| |
770 | 770 |
| |
771 | 771 |
| |
772 |
| - | |
773 |
| - | |
| 772 | + | |
| 773 | + | |
| 774 | + | |
| 775 | + | |
| 776 | + | |
774 | 777 |
| |
775 | 778 |
| |
776 | 779 |
| |
|
Lines changed: 7 additions & 5 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1402 | 1402 |
| |
1403 | 1403 |
| |
1404 | 1404 |
| |
1405 |
| - | |
1406 |
| - | |
1407 |
| - | |
1408 |
| - | |
1409 |
| - | |
| 1405 | + | |
| 1406 | + | |
| 1407 | + | |
| 1408 | + | |
| 1409 | + | |
| 1410 | + | |
| 1411 | + | |
1410 | 1412 |
| |
1411 | 1413 |
| |
1412 | 1414 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
504 | 504 |
| |
505 | 505 |
| |
506 | 506 |
| |
507 |
| - | |
| 507 | + | |
508 | 508 |
| |
509 | 509 |
| |
510 | 510 |
| |
|
0 commit comments
Comments
(0)