partial responses in endpoints_webapp2.py |
|||
Issue descriptionGoogle APIs support a magical query string parameter "fields". Value format docs: https://cloud.google.com/storage/docs/json_api/v1/how-tos/performance#partial-response Feature request: add support for "fields" into endpoints_webapp2.py When fields are specified, strip all other fields from the message returned by the service implementation. Only after that, continue to serializing the message to JSON and responding to the requester.
,
May 22 2018
,
Jun 15 2018
The following revision refers to this bug: https://chromium.googlesource.com/infra/luci/luci-py.git/+/5aeacfea2e39e4d65560b865757d96329c4b01c8 commit 5aeacfea2e39e4d65560b865757d96329c4b01c8 Author: smut <smut@google.com> Date: Fri Jun 15 21:14:52 2018 [webapp2 adapter] Parse fields partial response strings Bug: 833882 Change-Id: I53d305a41904f46e38183d96a6466b7136fd210b Reviewed-on: https://chromium-review.googlesource.com/1101919 Commit-Queue: smut <smut@google.com> Reviewed-by: Nodir Turakulov <nodir@chromium.org> [add] https://crrev.com/5aeacfea2e39e4d65560b865757d96329c4b01c8/appengine/components/components/endpoints_webapp2/partial.py [add] https://crrev.com/5aeacfea2e39e4d65560b865757d96329c4b01c8/appengine/components/components/endpoints_webapp2/partial_test.py
,
Jul 18
The following revision refers to this bug: https://chromium.googlesource.com/infra/luci/luci-py.git/+/b7a967cc0c8199b96cbf61e393a90e4a6d7c9748 commit b7a967cc0c8199b96cbf61e393a90e4a6d7c9748 Author: smut <smut@google.com> Date: Wed Jul 18 21:28:26 2018 [webapp2 adapter] Support application of partial response masks Bug: 833882 Change-Id: I16fa336d2cbfe716479f28a428b81d2a3d7906c9 Reviewed-on: https://chromium-review.googlesource.com/1134410 Commit-Queue: smut <smut@google.com> Reviewed-by: Nodir Turakulov <nodir@chromium.org> [modify] https://crrev.com/b7a967cc0c8199b96cbf61e393a90e4a6d7c9748/appengine/components/components/endpoints_webapp2/partial.py [modify] https://crrev.com/b7a967cc0c8199b96cbf61e393a90e4a6d7c9748/appengine/components/components/endpoints_webapp2/partial_test.py
,
Jul 25
The following revision refers to this bug: https://chromium.googlesource.com/infra/luci/luci-py.git/+/c5479d4bee0939d1d6401dc51029b1ec5a084870 commit c5479d4bee0939d1d6401dc51029b1ec5a084870 Author: smut <smut@google.com> Date: Wed Jul 25 23:26:35 2018 [webapp2 adapter] Return partial responses Bug: 833882 Change-Id: Ib387c787d2843541b8e3afa581907ac0d429199d Reviewed-on: https://chromium-review.googlesource.com/1149158 Commit-Queue: smut <smut@google.com> Reviewed-by: Nodir Turakulov <nodir@chromium.org> [modify] https://crrev.com/c5479d4bee0939d1d6401dc51029b1ec5a084870/appengine/components/components/endpoints_webapp2/adapter.py
,
Jul 26
,
Jul 26
🎉
,
Aug 1
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/b95687a07fc680eb1fe0e8d8f976167e22b53bb9 commit b95687a07fc680eb1fe0e8d8f976167e22b53bb9 Author: smut <smut@google.com> Date: Wed Aug 01 01:45:47 2018 Roll luci-py to db606474e98248472fbfc9a2278f94380e31c07d Changes: db606474 [Swarming] Ensure lease_expiration_ts is not None before comparing 920af4af cq.cfg: remove cq_name. 47e502ce Reland "Send spec_name tags to event mon metrics" d1d13040 [Swarming] Update assertion to account for indefinite leases de2166f3 [Swarming] Account for unspecified leased_indefinitely value c5479d4b [webapp2 adapter] Return partial responses ac302121 [swarming] start_server.py add --leak and print log path fcc3c00e [swarming] improve README for local dev be667854 [client] Overhaul temporary directory override in run_isolated e7c676c3 [Swarming] Handle MP release failure during bot delete API call a41b71fa [Swarming] Support indefinite leases ed2e0acd README.md: clarify OAuth instructions for developer instance 55e57661 [swarming] fix local_smoke_test a1a5e671 [client] lower default task priority to 200 87242d7b Roll infra_libs, gae_event_mon and gae_ts_mon. dd874b59 [Machine Provider] Fix lint errors and enable pylint presubmit c9c4e0a0 [Machine Provider] Support indefinite leases Bug: 833882 Change-Id: I7d734abc57dc189f80d461382ecacecf8bc07a4c Reviewed-on: https://chromium-review.googlesource.com/1157660 Reviewed-by: Marc-Antoine Ruel <maruel@chromium.org> Commit-Queue: smut <smut@google.com> [modify] https://crrev.com/b95687a07fc680eb1fe0e8d8f976167e22b53bb9/DEPS |
|||
►
Sign in to add a comment |
|||
Comment 1 by no...@chromium.org
, Apr 17 2018