Commit 428831b
committed
[Flight Reply] Reject any new Chunks not yet discovered at the time of reportGlobalError (#31840)
We might have already resolved models that are not pending and so are
not rejected by aborting the stream. When those later get parsed they
might discover new chunks which end up as pending. These should be
errored since they will never be able to resolve later.
This avoids infinitely hanging the stream.
This same fix needs to be ported to ReactFlightClient that has the same
issue.
DiffTrain build for [ef979d4](ef979d4)1 parent f2bb2d4 commit 428831b
File tree
24 files changed
+97
-108
lines changed- compiled-rn
- facebook-fbsource/xplat/js
- RKJSModules/vendor/react
- react-dom/cjs
- react-test-renderer/cjs
- react/cjs
- scheduler/cjs
- react-native-github/Libraries/Renderer
- implementations
24 files changed
+97
-108
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
420 | 420 | | |
421 | 421 | | |
422 | 422 | | |
423 | | - | |
| 423 | + | |
424 | 424 | | |
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
203 | 203 | | |
204 | 204 | | |
205 | 205 | | |
206 | | - | |
| 206 | + | |
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
203 | 203 | | |
204 | 204 | | |
205 | 205 | | |
206 | | - | |
| 206 | + | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
25852 | 25852 | | |
25853 | 25853 | | |
25854 | 25854 | | |
25855 | | - | |
| 25855 | + | |
25856 | 25856 | | |
25857 | 25857 | | |
25858 | 25858 | | |
25859 | | - | |
| 25859 | + | |
25860 | 25860 | | |
25861 | 25861 | | |
25862 | 25862 | | |
| |||
25893 | 25893 | | |
25894 | 25894 | | |
25895 | 25895 | | |
25896 | | - | |
| 25896 | + | |
25897 | 25897 | | |
25898 | 25898 | | |
25899 | | - | |
| 25899 | + | |
25900 | 25900 | | |
25901 | 25901 | | |
25902 | 25902 | | |
| |||
26042 | 26042 | | |
26043 | 26043 | | |
26044 | 26044 | | |
26045 | | - | |
| 26045 | + | |
26046 | 26046 | | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
15869 | 15869 | | |
15870 | 15870 | | |
15871 | 15871 | | |
15872 | | - | |
| 15872 | + | |
15873 | 15873 | | |
15874 | 15874 | | |
15875 | 15875 | | |
15876 | 15876 | | |
15877 | 15877 | | |
15878 | 15878 | | |
15879 | | - | |
| 15879 | + | |
15880 | 15880 | | |
15881 | 15881 | | |
15882 | 15882 | | |
| |||
15898 | 15898 | | |
15899 | 15899 | | |
15900 | 15900 | | |
15901 | | - | |
| 15901 | + | |
15902 | 15902 | | |
15903 | 15903 | | |
15904 | | - | |
| 15904 | + | |
15905 | 15905 | | |
15906 | 15906 | | |
15907 | 15907 | | |
| |||
16007 | 16007 | | |
16008 | 16008 | | |
16009 | 16009 | | |
16010 | | - | |
| 16010 | + | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
16522 | 16522 | | |
16523 | 16523 | | |
16524 | 16524 | | |
16525 | | - | |
| 16525 | + | |
16526 | 16526 | | |
16527 | 16527 | | |
16528 | 16528 | | |
16529 | 16529 | | |
16530 | 16530 | | |
16531 | 16531 | | |
16532 | | - | |
| 16532 | + | |
16533 | 16533 | | |
16534 | 16534 | | |
16535 | 16535 | | |
| |||
16551 | 16551 | | |
16552 | 16552 | | |
16553 | 16553 | | |
16554 | | - | |
| 16554 | + | |
16555 | 16555 | | |
16556 | 16556 | | |
16557 | | - | |
| 16557 | + | |
16558 | 16558 | | |
16559 | 16559 | | |
16560 | 16560 | | |
| |||
16675 | 16675 | | |
16676 | 16676 | | |
16677 | 16677 | | |
16678 | | - | |
| 16678 | + | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
25913 | 25913 | | |
25914 | 25914 | | |
25915 | 25915 | | |
25916 | | - | |
| 25916 | + | |
25917 | 25917 | | |
25918 | 25918 | | |
25919 | 25919 | | |
25920 | | - | |
| 25920 | + | |
25921 | 25921 | | |
25922 | 25922 | | |
25923 | 25923 | | |
| |||
25954 | 25954 | | |
25955 | 25955 | | |
25956 | 25956 | | |
25957 | | - | |
| 25957 | + | |
25958 | 25958 | | |
25959 | 25959 | | |
25960 | | - | |
| 25960 | + | |
25961 | 25961 | | |
25962 | 25962 | | |
25963 | 25963 | | |
| |||
26419 | 26419 | | |
26420 | 26420 | | |
26421 | 26421 | | |
26422 | | - | |
| 26422 | + | |
26423 | 26423 | | |
26424 | 26424 | | |
26425 | 26425 | | |
| |||
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
15880 | 15880 | | |
15881 | 15881 | | |
15882 | 15882 | | |
15883 | | - | |
| 15883 | + | |
15884 | 15884 | | |
15885 | 15885 | | |
15886 | 15886 | | |
15887 | 15887 | | |
15888 | 15888 | | |
15889 | 15889 | | |
15890 | | - | |
| 15890 | + | |
15891 | 15891 | | |
15892 | 15892 | | |
15893 | 15893 | | |
| |||
15909 | 15909 | | |
15910 | 15910 | | |
15911 | 15911 | | |
15912 | | - | |
| 15912 | + | |
15913 | 15913 | | |
15914 | 15914 | | |
15915 | | - | |
| 15915 | + | |
15916 | 15916 | | |
15917 | 15917 | | |
15918 | 15918 | | |
| |||
16172 | 16172 | | |
16173 | 16173 | | |
16174 | 16174 | | |
16175 | | - | |
| 16175 | + | |
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
16537 | 16537 | | |
16538 | 16538 | | |
16539 | 16539 | | |
16540 | | - | |
| 16540 | + | |
16541 | 16541 | | |
16542 | 16542 | | |
16543 | 16543 | | |
16544 | 16544 | | |
16545 | 16545 | | |
16546 | 16546 | | |
16547 | | - | |
| 16547 | + | |
16548 | 16548 | | |
16549 | 16549 | | |
16550 | 16550 | | |
| |||
16566 | 16566 | | |
16567 | 16567 | | |
16568 | 16568 | | |
16569 | | - | |
| 16569 | + | |
16570 | 16570 | | |
16571 | 16571 | | |
16572 | | - | |
| 16572 | + | |
16573 | 16573 | | |
16574 | 16574 | | |
16575 | 16575 | | |
| |||
16844 | 16844 | | |
16845 | 16845 | | |
16846 | 16846 | | |
16847 | | - | |
| 16847 | + | |
16848 | 16848 | | |
16849 | 16849 | | |
16850 | 16850 | | |
| |||
0 commit comments