Skip to content

bigquery: proper handle options to Job.getQueryResults#2713

Merged
pongad merged 2 commits intogoogleapis:bq-perffrom
pongad:bq-opt
Dec 20, 2017
Merged

bigquery: proper handle options to Job.getQueryResults#2713
pongad merged 2 commits intogoogleapis:bq-perffrom
pongad:bq-opt

Conversation

@pongad
Copy link
Contributor

@pongad pongad commented Dec 15, 2017

No description provided.

@pongad pongad requested a review from tswast December 15, 2017 07:34
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 15, 2017

List<QueryResultsOption> waitOptions = new ArrayList<>();
waitOptions.add(QueryResultsOption.pageSize(0L));
List<TableDataListOption> listOptions = new ArrayList<>();

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.


List<QueryResultsOption> waitOptions = new ArrayList<>();
waitOptions.add(QueryResultsOption.pageSize(0L));
List<TableDataListOption> listOptions = new ArrayList<>();

This comment was marked as spam.

}
}
if (!hasWaitTime) {
waitOptions.add(QueryResultsOption.maxWaitTime(Duration.ofMinutes(1).toMillis()));

This comment was marked as spam.

This comment was marked as spam.

@pongad pongad requested a review from jba December 19, 2017 07:38
@pongad pongad merged commit c34e949 into googleapis:bq-perf Dec 20, 2017
@pongad pongad deleted the bq-opt branch December 20, 2017 04:38
chingor13 pushed a commit that referenced this pull request Feb 20, 2026
#2713)

* chore: Update generation configuration at Sat Oct 12 02:22:47 UTC 2024

* chore: generate libraries at Sat Oct 12 02:23:08 UTC 2024

* chore: Update generation configuration at Sun Oct 13 02:27:02 UTC 2024
chingor13 pushed a commit that referenced this pull request Mar 24, 2026
In this PR:
- Remove test output directory.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants