bigquery: fix error callback arguments #737
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
callmehiphop
added
type: bug
googlebot
added
the
cla: yes
Contributor
|
Good catch! Could you also show that the apiResponse comes back in the @example? |
Dataset#setMetadata was not passing the correct arguments in for request failures Updated setMetadata example to reflect callback arguments
Contributor
Author
|
@stephenplusplus done! |
stephenplusplus
added a commit
that referenced
this pull request
Jul 23, 2015
bigquery: fix error callback arguments
Contributor
|
Thanks! |
4 tasks
sofisl
pushed a commit
that referenced
this pull request
Nov 11, 2022
4 tasks
sofisl
pushed a commit
that referenced
this pull request
Jan 10, 2023
* chore(main): release 4.1.1 * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
miguelvelezsa
pushed a commit
that referenced
this pull request
Jul 23, 2025
BREAKING CHANGE It won't be possible to pass a third-party Promise constructor (e.g. bluebird promise) to gax-managed client libraries.
miguelvelezsa
pushed a commit
that referenced
this pull request
Jul 23, 2025
🤖 I have created a release \*beep\* \*boop\* --- ## [2.0.0](https://www.github.com/googleapis/gax-nodejs/compare/v1.14.2...v2.0.0) (2020-03-26) ### ⚠ BREAKING CHANGES * use Node.js v10+ * throw for versions of Node.js older than v10.0.0 (#748) * stop accepting Promise constructor (#737) ### Features * export bundle descriptor in descriptors interface ([#744](https://www.github.com/googleapis/gax-nodejs/issues/744)) ([b1eccf9](https://www.github.com/googleapis/gax-nodejs/commit/b1eccf96c439d67376d249a54c9d22ffe7ff1839)) * export ServiceError from @grpc/grpc-js ([#754](https://www.github.com/googleapis/gax-nodejs/issues/754)) ([24a4d60](https://www.github.com/googleapis/gax-nodejs/commit/24a4d600738a9597e9a87d6705eaed3dc2285e3b)) * stop accepting Promise constructor ([#737](https://www.github.com/googleapis/gax-nodejs/issues/737)) ([816bf9b](https://www.github.com/googleapis/gax-nodejs/commit/816bf9b283217208debd979e893a6daf29f1f739)) * support async iterator for paging method ([#708](https://www.github.com/googleapis/gax-nodejs/issues/708)) ([3ac5afb](https://www.github.com/googleapis/gax-nodejs/commit/3ac5afb3b1b1b22798f15ee07395f3ca765383b4)) * throw for versions of Node.js older than v10.0.0 ([#748](https://www.github.com/googleapis/gax-nodejs/issues/748)) ([511fc23](https://www.github.com/googleapis/gax-nodejs/commit/511fc233bd66d537c24743ef460ee8c609cd551f)) ### Bug Fixes * **deps:** update dependency @grpc/grpc-js to ^0.7.0 ([#736](https://www.github.com/googleapis/gax-nodejs/issues/736)) ([01c428c](https://www.github.com/googleapis/gax-nodejs/commit/01c428cb1240320b92778abf1297a5ff72346fd9)) * **deps:** use @grpc/grpc-js v0.7.2 ([#735](https://www.github.com/googleapis/gax-nodejs/issues/735)) ([836e81b](https://www.github.com/googleapis/gax-nodejs/commit/836e81b64f84d8c118e6aea0580f0645658a8490)) * **deps:** use protobuf.js v6.8.9 ([#743](https://www.github.com/googleapis/gax-nodejs/issues/743)) ([fab91ce](https://www.github.com/googleapis/gax-nodejs/commit/fab91ce334d76212d7e31b5478331339c5acad76)) * allow passing numbers as path template parameters ([#756](https://www.github.com/googleapis/gax-nodejs/issues/756)) ([c466d3d](https://www.github.com/googleapis/gax-nodejs/commit/c466d3dc68c8f9050d3ae69dcedd708e3509ae17)) ### Miscellaneous Chores * require Node.js v10+ ([#759](https://www.github.com/googleapis/gax-nodejs/issues/759)) ([23ec7f6](https://www.github.com/googleapis/gax-nodejs/commit/23ec7f69c3813f6d06ea8b2a473d072337e1b499)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please).
miguelvelezsa
pushed a commit
that referenced
this pull request
Jul 23, 2025
* chore: use gts v2 (#757) googleapis/gax-nodejs@247aeb7 commit 247aeb7668c4787596aeaea22f727549bcdf068d Author: Alexander Fenster <[email protected]> Date: Thu Mar 26 13:04:10 2020 -0700 chore: use gts v2 (#757) * chore: use gts v2 * chore: ahhhhh thaaaats why * chore: do not run on node8 * fix: new gts rules * build: use TypeScript ^3.8.3, set lib to es2018 (#760) googleapis/gax-nodejs@41c73f8 commit 41c73f886e0106d0d7b1b851b4823990495d8fc7 Author: Alexander Fenster <[email protected]> Date: Thu Mar 26 13:34:19 2020 -0700 build: use TypeScript ^3.8.3, set lib to es2018 (#760) * build: use TypeScript ^3.8.3, set lib to es2018 * fix: better typing * chore!: require Node.js v10+ (#759) googleapis/gax-nodejs@23ec7f6 commit 23ec7f69c3813f6d06ea8b2a473d072337e1b499 Author: Alexander Fenster <[email protected]> Date: Thu Mar 26 14:07:14 2020 -0700 chore!: require Node.js v10+ (#759) BREAKING CHANGE: use Node.js v10+ * chore(deps): update dependency @types/rimraf to v3 (#751) googleapis/gax-nodejs@2f1b6ae commit 2f1b6aedbbd620a1e9872b858e58264056344571 Author: WhiteSource Renovate <[email protected]> Date: Thu Mar 26 22:27:20 2020 +0100 chore(deps): update dependency @types/rimraf to v3 (#751) Co-authored-by: Alexander Fenster <[email protected]> * chore: release 2.0.0 (#738) googleapis/gax-nodejs@1c11c80 commit 1c11c8015f93a6a711a9f113105d027095a3d90c Author: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Date: Thu Mar 26 22:32:20 2020 +0000 chore: release 2.0.0 (#738) 🤖 I have created a release \*beep\* \*boop\* --- ## [2.0.0](https://www.github.com/googleapis/gax-nodejs/compare/v1.14.2...v2.0.0) (2020-03-26) ### ⚠ BREAKING CHANGES * use Node.js v10+ * throw for versions of Node.js older than v10.0.0 (#748) * stop accepting Promise constructor (#737) ### Features * export bundle descriptor in descriptors interface ([#744](https://www.github.com/googleapis/gax-nodejs/issues/744)) ([b1eccf9](https://www.github.com/googleapis/gax-nodejs/commit/b1eccf96c439d67376d249a54c9d22ffe7ff1839)) * export ServiceError from @grpc/grpc-js ([#754](https://www.github.com/googleapis/gax-nodejs/issues/754)) ([24a4d60](https://www.github.com/googleapis/gax-nodejs/commit/24a4d600738a9597e9a87d6705eaed3dc2285e3b)) * stop accepting Promise constructor ([#737](https://www.github.com/googleapis/gax-nodejs/issues/737)) ([816bf9b](https://www.github.com/googleapis/gax-nodejs/commit/816bf9b283217208debd979e893a6daf29f1f739)) * support async iterator for paging method ([#708](https://www.github.com/googleapis/gax-nodejs/issues/708)) ([3ac5afb](https://www.github.com/googleapis/gax-nodejs/commit/3ac5afb3b1b1b22798f15ee07395f3ca765383b4)) * throw for versions of Node.js older than v10.0.0 ([#748](https://www.github.com/googleapis/gax-nodejs/issues/748)) ([511fc23](https://www.github.com/googleapis/gax-nodejs/commit/511fc233bd66d537c24743ef460ee8c609cd551f)) ### Bug Fixes * **deps:** update dependency @grpc/grpc-js to ^0.7.0 ([#736](https://www.github.com/googleapis/gax-nodejs/issues/736)) ([01c428c](https://www.github.com/googleapis/gax-nodejs/commit/01c428cb1240320b92778abf1297a5ff72346fd9)) * **deps:** use @grpc/grpc-js v0.7.2 ([#735](https://www.github.com/googleapis/gax-nodejs/issues/735)) ([836e81b](https://www.github.com/googleapis/gax-nodejs/commit/836e81b64f84d8c118e6aea0580f0645658a8490)) * **deps:** use protobuf.js v6.8.9 ([#743](https://www.github.com/googleapis/gax-nodejs/issues/743)) ([fab91ce](https://www.github.com/googleapis/gax-nodejs/commit/fab91ce334d76212d7e31b5478331339c5acad76)) * allow passing numbers as path template parameters ([#756](https://www.github.com/googleapis/gax-nodejs/issues/756)) ([c466d3d](https://www.github.com/googleapis/gax-nodejs/commit/c466d3dc68c8f9050d3ae69dcedd708e3509ae17)) ### Miscellaneous Chores * require Node.js v10+ ([#759](https://www.github.com/googleapis/gax-nodejs/issues/759)) ([23ec7f6](https://www.github.com/googleapis/gax-nodejs/commit/23ec7f69c3813f6d06ea8b2a473d072337e1b499)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). * fix(deps): update dependency google-auth-library to v6 (#763) googleapis/gax-nodejs@72114db commit 72114db1b15af3034c51a784e3fc619e2ee281e5 Author: WhiteSource Renovate <[email protected]> Date: Sat Mar 28 04:08:38 2020 +0100 fix(deps): update dependency google-auth-library to v6 (#763) * fix: make async iteration work for gRPC-fallback; refactor the code (#765) googleapis/gax-nodejs@944c06b commit 944c06b4225c4dd5cdcf08e4ca2497cfe3a69cde Author: Alexander Fenster <[email protected]> Date: Fri Mar 27 22:19:15 2020 -0700 fix: make async iteration work for gRPC-fallback; refactor the code (#765) * chore: release 2.0.1 (#766) googleapis/gax-nodejs@3ce902d commit 3ce902d94f6112853df4098b50ca46bb228a69b5 Author: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Date: Sat Mar 28 05:30:07 2020 +0000 chore: release 2.0.1 (#766) 🤖 I have created a release \*beep\* \*boop\* --- ### [2.0.1](https://www.github.com/googleapis/gax-nodejs/compare/v2.0.0...v2.0.1) (2020-03-28) ### Bug Fixes * **deps:** update dependency google-auth-library to v6 ([#763](https://www.github.com/googleapis/gax-nodejs/issues/763)) ([72114db](https://www.github.com/googleapis/gax-nodejs/commit/72114db1b15af3034c51a784e3fc619e2ee281e5)) * make async iteration work for gRPC-fallback; refactor the code ([#765](https://www.github.com/googleapis/gax-nodejs/issues/765)) ([944c06b](https://www.github.com/googleapis/gax-nodejs/commit/944c06b4225c4dd5cdcf08e4ca2497cfe3a69cde)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). * build: set AUTOSYNTH_MULTIPLE_COMMITS=true for context aware commits (#770) googleapis/gax-nodejs@a1e4a3f commit a1e4a3fd9877b1370869a919e1165ea69410f201 Author: Benjamin E. Coe <[email protected]> Date: Tue Mar 31 18:34:30 2020 -0700 build: set AUTOSYNTH_MULTIPLE_COMMITS=true for context aware commits (#770) * fix: do not run node 8 CI (#456) googleapis/synthtool@1b4cc80 commit 1b4cc80a7aaf164f6241937dd87f3bd1f4149e0c Author: Alexander Fenster <[email protected]> Date: Wed Mar 25 08:01:31 2020 -0700 fix: do not run node 8 CI (#456) * fix: update template files for Node.js libraries (#463) googleapis/synthtool@9982024 commit 99820243d348191bc9c634f2b48ddf65096285ed Author: Alexander Fenster <[email protected]> Date: Tue Mar 31 11:56:27 2020 -0700 fix: update template files for Node.js libraries (#463)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Dataset#setMetadatawas not passing the correct arguments in for request failures