Retry for 'mutate_rows' that return retryable errors.#4256

Merged

lukesneeringer merged 17 commits intogoogleapis:masterfrom

justinseinlin:mutate-rows-retry-2

Nov 2, 2017

Commits on Oct 25, 2017

Retry for rows that return retryable error codes.
justinseinlin
committed
Stop processing once all rows are successful or non-retryable.
justinseinlin
committed
Remove retry for INTERNAL error. Fix the case where has_retryable_responses can get overridden by non-retryableresponse.
justinseinlin
committed
Fix lint errors.
justinseinlin
committed
Add tests for google.cloud.bigtable.table.Table.mutate_rows.\nCases for 1) no retry 2) one retry 3) timeout.
justinseinlin
committed
Add test cases for _RetryableMutateRowsWorker.
justinseinlin
committed
Refactor test for _RetryableMutateRowsWorker to cover callable interface.
justinseinlin
committed
Remove assumption that responses come back in the same linear order as request rows.
justinseinlin
committed
Include retryable responses in the exception raised.
justinseinlin
committed
Assign default value for init argument.
justinseinlin
committed
Change _MILLIS_PER_SECOND to float for better deadline precision.
justinseinlin
committed
Use google.api.core.retry.Retry. Allow user-configurable retry-options.
justinseinlin
committed
Revert retry.py. Modify mutate_rows api to take a Retry instead with a DEFAULT RETRY.
justinseinlin
committed
Use google.api.core.exceptions for internal retryable worker instead of creating a custom exception. Also update documentation.
justinseinlin
committed
Add sanity check for rows requested to mutate and number of responses coming back.
justinseinlin
committed
Add test for assertion check to ensure num of responses matches num of rows requested.
justinseinlin
committed
Refactor google.api.core to google.api_core.
justinseinlin
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Retry for 'mutate_rows' that return retryable errors.#4256

Retry for 'mutate_rows' that return retryable errors.#4256
lukesneeringer merged 17 commits intogoogleapis:masterfrom
justinseinlin:mutate-rows-retry-2

Commits on Oct 25, 2017

Retry for rows that return retryable error codes.

Stop processing once all rows are successful or non-retryable.

Remove retry for INTERNAL error. Fix the case where has_retryable_responses can get overridden by non-retryableresponse.

Fix lint errors.

Add tests for google.cloud.bigtable.table.Table.mutate_rows.\nCases for 1) no retry 2) one retry 3) timeout.

Add test cases for _RetryableMutateRowsWorker.

Refactor test for _RetryableMutateRowsWorker to cover callable interface.

Remove assumption that responses come back in the same linear order as request rows.

Include retryable responses in the exception raised.

Assign default value for init argument.

Change _MILLIS_PER_SECOND to float for better deadline precision.

Use google.api.core.retry.Retry. Allow user-configurable retry-options.

Revert retry.py. Modify mutate_rows api to take a Retry instead with a DEFAULT RETRY.

Use google.api.core.exceptions for internal retryable worker instead of creating a custom exception. Also update documentation.

Add sanity check for rows requested to mutate and number of responses coming back.

Add test for assertion check to ensure num of responses matches num of rows requested.

Refactor google.api.core to google.api_core.

Retry for 'mutate_rows' that return retryable errors.#4256lukesneeringer merged 17 commits intogoogleapis:mastergoogleapis/google-cloud-python:masterfrom justinseinlin:mutate-rows-retry-2justinseinlin/google-cloud-python:mutate-rows-retry-2Copy head branch name to clipboard

Commits

Commits on Oct 25, 2017

Retry for 'mutate_rows' that return retryable errors.#4256
lukesneeringer merged 17 commits intogoogleapis:masterfrom
justinseinlin:mutate-rows-retry-2