Project

General

Profile

Actions

Feature #18215

closed

Use 'select' on collection updates to avoid returning manifest_text

Added by Peter Amstutz over 2 years ago. Updated over 2 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Story points:
-
Release relationship:
Auto

Description

On the wb2 collection service, use select so that the update only returns the uuid field and not the manifest_text field in the response.


Subtasks 1 (0 open1 closed)

Arvados - Task #18355: Review 18215-collection-update-without-manifestResolvedStephen Smith11/16/2021Actions

Related issues

Related to Arvados - Bug #18122: Support 'select' on 'get', 'update', and 'delete' calls to limit what fields are returned in update responseResolvedTom Clegg09/16/2021Actions
Actions #1

Updated by Peter Amstutz over 2 years ago

  • Target version set to 2021-10-27 sprint
Actions #2

Updated by Peter Amstutz over 2 years ago

  • Target version changed from 2021-10-27 sprint to 2021-11-10 sprint
  • Category deleted (Workbench2)
Actions #3

Updated by Peter Amstutz over 2 years ago

  • Target version changed from 2021-11-10 sprint to 2021-11-24 sprint
Actions #4

Updated by Tom Clegg over 2 years ago

  • Related to Bug #18122: Support 'select' on 'get', 'update', and 'delete' calls to limit what fields are returned in update response added
Actions #5

Updated by Peter Amstutz over 2 years ago

  • Description updated (diff)
Actions #6

Updated by Lucas Di Pentima over 2 years ago

  • Assigned To set to Lucas Di Pentima
Actions #7

Updated by Lucas Di Pentima over 2 years ago

  • Status changed from New to In Progress
Actions #8

Updated by Lucas Di Pentima over 2 years ago

Updates at commit:a969c0d3e - branch 18215-select-param-update-create
Test run: developer-run-tests: #2794

  • Makes controller to just respond with selected field on create & update calls.
Actions #9

Updated by Tom Clegg over 2 years ago

18215-select-param-update-create LGTM - maybe worth adding Create and Update calls to (*RouterIntegrationSuite) TestSelectParam()?

Actions #10

Updated by Lucas Di Pentima over 2 years ago

Updates at arvados-workbench2|e3655d66 - branch 18215-collection-update-without-manifest
Test run: developer-tests-workbench2: #510

  • Updates commonResourceService.update() so that it accepts an optional select param.
  • Updates collectionService.update() so that it automatically passes a select parameter with the list of updated fields, plus some other interesting ones like version & modified_at.
  • Modifies code handling the collection update() responses so that it updates the stored cache data instead of replacing it.
  • Adds test.
Actions #11

Updated by Lucas Di Pentima over 2 years ago

Update at arvados-workbench2|cb5f37e7
Test run: developer-tests-workbench2: #511

Actions #12

Updated by Stephen Smith over 2 years ago

This looks good to me

Actions #13

Updated by Lucas Di Pentima over 2 years ago

  • Status changed from In Progress to Resolved
Actions #14

Updated by Peter Amstutz over 2 years ago

  • Release set to 45
Actions

Also available in: Atom PDF