Commit b692b9d
committed
core: Handle NR/LB exceptions when panicking
If a panic is followed a panic, we'd ignore the second. But if an
exception happens while entering panic mode we may fail to update the
picker with the first error. This is "fine" from a correctness
standpoint; all bets are off when panicking and we've already logged the
first error. But failing RPCs can often be more easily seen than just
the log.
Noticed because of http://yaqs/84937855986858721281 parent 99f8683 commit b692b9d
File tree
1 file changed
+10
-24
lines changed- core/src/main/java/io/grpc/internal
1 file changed
+10
-24
lines changedLines changed: 10 additions & 24 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
779 | 779 | | |
780 | 780 | | |
781 | 781 | | |
782 | | - | |
783 | | - | |
784 | | - | |
785 | | - | |
786 | | - | |
787 | | - | |
788 | | - | |
789 | | - | |
790 | | - | |
791 | | - | |
792 | | - | |
793 | | - | |
794 | | - | |
795 | | - | |
796 | | - | |
797 | | - | |
798 | | - | |
799 | | - | |
| 782 | + | |
| 783 | + | |
| 784 | + | |
| 785 | + | |
| 786 | + | |
| 787 | + | |
| 788 | + | |
| 789 | + | |
| 790 | + | |
800 | 791 | | |
801 | | - | |
802 | | - | |
803 | | - | |
804 | | - | |
805 | | - | |
806 | 792 | | |
807 | 793 | | |
808 | 794 | | |
| |||
1404 | 1390 | | |
1405 | 1391 | | |
1406 | 1392 | | |
1407 | | - | |
| 1393 | + | |
1408 | 1394 | | |
1409 | 1395 | | |
1410 | 1396 | | |
| |||
0 commit comments