Commit f772f86
committed
feat(generator): emit resumable upload settings and HttpJson upload stub
Adds generator support for protocol-specific resumable upload options, independent of any transport stub wiring:
- Emit ResumableUploadCallSettings on the generated <Service>Settings and
<Service>StubSettings, populated with global timeouts extracted from the
service config while bypassing standard unary retries.
- Add HttpJsonServiceResumableUploadStubClassComposer, which generates the
dedicated REST stub (HttpJson[Service]ResumableUploadStub) for services
containing resumable upload RPC methods.
- Extract HttpJsonDescriptorComposer out of HttpJsonServiceStubClassComposer
and add path prefix support. Pure refactor with no golden changes, reused by
the upload stub composer above.
Supersedes #14318 and #14320, which are folded in here.1 parent acc1856 commit f772f86
18 files changed
Lines changed: 3195 additions & 739 deletions
File tree
- sdk-platform-java/gapic-generator-java/src
- main/java/com/google/api/generator/gapic/composer
- comment
- common
- rest
- samplecode
- test/java/com/google/api/generator/gapic/composer
- common
- grpc
- goldens
- rest
- goldens
Lines changed: 34 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
130 | 130 | | |
131 | 131 | | |
132 | 132 | | |
| 133 | + | |
| 134 | + | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
133 | 143 | | |
134 | 144 | | |
135 | 145 | | |
| |||
140 | 150 | | |
141 | 151 | | |
142 | 152 | | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
| 160 | + | |
| 161 | + | |
| 162 | + | |
| 163 | + | |
| 164 | + | |
| 165 | + | |
| 166 | + | |
| 167 | + | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
| 173 | + | |
| 174 | + | |
| 175 | + | |
| 176 | + | |
143 | 177 | | |
144 | 178 | | |
145 | 179 | | |
| |||
Lines changed: 31 additions & 11 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
| 28 | + | |
28 | 29 | | |
29 | 30 | | |
30 | 31 | | |
| |||
153 | 154 | | |
154 | 155 | | |
155 | 156 | | |
156 | | - | |
| 157 | + | |
157 | 158 | | |
158 | 159 | | |
159 | 160 | | |
| |||
311 | 312 | | |
312 | 313 | | |
313 | 314 | | |
314 | | - | |
315 | | - | |
316 | | - | |
| 315 | + | |
| 316 | + | |
| 317 | + | |
| 318 | + | |
| 319 | + | |
| 320 | + | |
| 321 | + | |
317 | 322 | | |
318 | 323 | | |
319 | | - | |
| 324 | + | |
| 325 | + | |
| 326 | + | |
320 | 327 | | |
321 | 328 | | |
322 | 329 | | |
| |||
794 | 801 | | |
795 | 802 | | |
796 | 803 | | |
| 804 | + | |
| 805 | + | |
| 806 | + | |
| 807 | + | |
| 808 | + | |
| 809 | + | |
| 810 | + | |
| 811 | + | |
| 812 | + | |
| 813 | + | |
797 | 814 | | |
798 | 815 | | |
799 | | - | |
800 | | - | |
801 | | - | |
802 | | - | |
803 | | - | |
| 816 | + | |
804 | 817 | | |
805 | 818 | | |
806 | 819 | | |
| |||
856 | 869 | | |
857 | 870 | | |
858 | 871 | | |
| 872 | + | |
859 | 873 | | |
860 | 874 | | |
861 | 875 | | |
| |||
933 | 947 | | |
934 | 948 | | |
935 | 949 | | |
936 | | - | |
| 950 | + | |
| 951 | + | |
| 952 | + | |
| 953 | + | |
| 954 | + | |
| 955 | + | |
| 956 | + | |
937 | 957 | | |
938 | 958 | | |
939 | 959 | | |
| |||
0 commit comments