Skip to content
This repository was archived by the owner on Nov 29, 2023. It is now read-only.

chore: change default retry policies timeouts (via synth) #48

Merged
merged 2 commits into from
Sep 16, 2020

Conversation

plamut
Copy link
Contributor

@plamut plamut commented Sep 15, 2020

This PR brings the generated parts of the code up to date, replacing several synth PRs that are quite old now (#35, #36, and #37).

The goal is to clean up the currently open PRs, make one last Python 2 compatible release, and then proceed with the migration to microgenerator.

@plamut
Copy link
Contributor Author

plamut commented Sep 15, 2020

Hmm, a few docstrings seem to have bad indentation, causing the docs to complain... will fix it.

@product-auto-label product-auto-label bot added the api: bigquerydatatransfer Issues related to the googleapis/python-bigquery-datatransfer API. label Sep 16, 2020
Copy link
Contributor

@tswast tswast left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple comments, but not seeing anything worth blocking this PR

requested_time_range (Union[dict, ~google.cloud.bigquery_datatransfer_v1.types.TimeRange]): Time range for the transfer runs that should be started.

If a dict is provided, it must be of the same form as the protobuf
message :class:`~google.cloud.bigquery_datatransfer_v1.types.TimeRange`
requested_run_time (Union[dict, ~google.cloud.bigquery_datatransfer_v1.types.Timestamp]): Specific run_time for a transfer run to be started. The
requested_run_time must not be in the future.
requested_run_time (Union[dict, ~google.cloud.bigquery_datatransfer_v1.types.Timestamp]): Protocol Buffers - Google's data interchange format Copyright 2008
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quite the docstring! Doesn't seem like the license should be part of an argument description, but I realize this is autogenned from proto source.

@leahecole leahecole requested review from leahecole and removed request for leahecole September 16, 2020 15:49
@leahecole
Copy link
Contributor

Not seeing anything super samples-y, and trust Bu Sun + Tim more than me on this one :)

@plamut plamut merged commit 78eea3c into googleapis:master Sep 16, 2020
@plamut plamut deleted the synth-update branch September 16, 2020 18:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: bigquerydatatransfer Issues related to the googleapis/python-bigquery-datatransfer API. 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