forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit52ad1e6
committed
Refactor jsonpath's compareDatetime()
This commit refactors come ridiculous coding in compareDatetime(). Also, itprovides correct cross-datatype comparison even when one of values overflowsduring cast. That eliminates dilemma on whether we should suppress overflowerrors during cast.Reported-by: Tom LaneDiscussion:https://postgr.es/m/32308.1569455803%40sss.pgh.pa.usDiscussion:https://postgr.es/m/a5629d0c-8162-7559-16aa-0c8390d6ba5f%40postgrespro.ruAuthor: Nikita Glukhov, Alexander Korotkov1 parenta6888fd commit52ad1e6
File tree
7 files changed
+237
-144
lines changed- src
- backend/utils/adt
- include/utils
- test/regress
- expected
- sql
7 files changed
+237
-144
lines changedLines changed: 24 additions & 16 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
554 | 554 |
| |
555 | 555 |
| |
556 | 556 |
| |
557 |
| - | |
558 |
| - | |
| 557 | + | |
| 558 | + | |
| 559 | + | |
559 | 560 |
| |
560 | 561 |
| |
561 |
| - | |
| 562 | + | |
562 | 563 |
| |
563 | 564 |
| |
564 | 565 |
| |
| |||
575 | 576 |
| |
576 | 577 |
| |
577 | 578 |
| |
578 |
| - | |
| 579 | + | |
579 | 580 |
| |
580 |
| - | |
| 581 | + | |
581 | 582 |
| |
582 | 583 |
| |
583 | 584 |
| |
| |||
596 | 597 |
| |
597 | 598 |
| |
598 | 599 |
| |
599 |
| - | |
| 600 | + | |
600 | 601 |
| |
601 | 602 |
| |
602 | 603 |
| |
603 | 604 |
| |
604 |
| - | |
| 605 | + | |
605 | 606 |
| |
606 | 607 |
| |
607 | 608 |
| |
608 | 609 |
| |
609 | 610 |
| |
610 |
| - | |
611 |
| - | |
| 611 | + | |
| 612 | + | |
| 613 | + | |
612 | 614 |
| |
613 | 615 |
| |
614 |
| - | |
| 616 | + | |
615 | 617 |
| |
616 | 618 |
| |
617 | 619 |
| |
| |||
631 | 633 |
| |
632 | 634 |
| |
633 | 635 |
| |
634 |
| - | |
| 636 | + | |
635 | 637 |
| |
636 |
| - | |
| 638 | + | |
637 | 639 |
| |
638 | 640 |
| |
639 | 641 |
| |
| |||
659 | 661 |
| |
660 | 662 |
| |
661 | 663 |
| |
662 |
| - | |
| 664 | + | |
663 | 665 |
| |
664 |
| - | |
| 666 | + | |
| 667 | + | |
| 668 | + | |
| 669 | + | |
| 670 | + | |
| 671 | + | |
| 672 | + | |
665 | 673 |
| |
666 | 674 |
| |
667 | 675 |
| |
| |||
677 | 685 |
| |
678 | 686 |
| |
679 | 687 |
| |
680 |
| - | |
| 688 | + | |
681 | 689 |
| |
682 | 690 |
| |
683 | 691 |
| |
684 | 692 |
| |
685 |
| - | |
| 693 | + | |
686 | 694 |
| |
687 | 695 |
| |
688 | 696 |
| |
|
0 commit comments
Comments
(0)