Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature: add support for 'fields' query parameter to getFiles #2138

Labels
api: storage Issues related to the googleapis/nodejs-storage API. priority: p3 Desirable enhancement or fix. May not be included in next release. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.

Comments

@ddelgrosso1
Copy link
Contributor

Currently getFiles retrieves all information / metadata about a File object. In some situations, for example deleting files, much of this information is unnecessary and can eat up memory if the list of files is large. Since getFiles is a precursor call for deleteFiles we should support the fields query string parameter so that the information returned can be reduced.

@ddelgrosso1 ddelgrosso1 added type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. priority: p3 Desirable enhancement or fix. May not be included in next release. labels Jan 31, 2023
@product-auto-label product-auto-label bot added the api: storage Issues related to the googleapis/nodejs-storage API. label Jan 31, 2023
thiyaguk09 added a commit to thiyaguk09/nodejs-storage-fork that referenced this issue Sep 16, 2024
…elds'-to-getFiles

feat(storage):  add support for 'fields' query parameter to getFiles
thiyaguk09 added a commit to thiyaguk09/nodejs-storage-fork that referenced this issue Sep 16, 2024
…elds'-to-getFiles

build: fix path-to-regexp to older version due to node 14 requirement
thiyaguk09 added a commit to thiyaguk09/nodejs-storage-fork that referenced this issue Sep 17, 2024
…elds'-to-getFiles

Remove unnecessary sample code
@danielduhh danielduhh reopened this Sep 17, 2024
@danielduhh
Copy link

We can close this out once #2521 is merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment