Remove docstring poetry by tseaver · Pull Request #413 · googleapis/google-cloud-python

Remove docstring poetry by tseaver · Pull Request #413 · googleapis/google-cloud-python

added 4 commits

December 10, 2014 12:47

Wrap prose docstring paragraphs at 72 characters.

tseaver added a commit that referenced this pull request

tseaver deleted the remove-docstring-poetry branch

December 11, 2014 05:08

parthea pushed a commit that referenced this pull request

* chore(main): release 2.3.0

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Holt Skinner <holtskinner@google.com>

parthea pushed a commit that referenced this pull request

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>

parthea pushed a commit that referenced this pull request

parthea pushed a commit that referenced this pull request

parthea pushed a commit that referenced this pull request

parthea pushed a commit that referenced this pull request

…mplates/python_library/.kokoro (#413)

Source-Link: googleapis/synthtool@30bd01b
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:9bc5fa3b62b091f60614c08a7fb4fd1d3e1678e326f34dd66ce1eefb5dc3267b

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

vchudnov-g pushed a commit that referenced this pull request

…le (#413)

Source-Link: googleapis/synthtool@a7ed11e
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:6e7328583be8edd3ba8f35311c76a1ecbc823010279ccb6ab46b7a76e25eafcc

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

parthea pushed a commit that referenced this pull request

parthea pushed a commit that referenced this pull request

parthea pushed a commit that referenced this pull request

parthea added a commit that referenced this pull request

parthea pushed a commit that referenced this pull request

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>

vchudnov-g pushed a commit that referenced this pull request

parthea pushed a commit that referenced this pull request

parthea added a commit that referenced this pull request

parthea pushed a commit that referenced this pull request

parthea pushed a commit that referenced this pull request

…or old behavior (#413)

* avoid parquet for older pandas

docs: deprecate `chunksize` when used with load jobs

* keep `chunksize` for future use in streaming APIs

deps: explicitly require `pyarrow >= 3.0`

* mention pyarrow as a dependency

* add pyarrow to conda deps

deps: explicitly require `numpy >= 1.16.6`

* update minimum numpy

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

parthea pushed a commit that referenced this pull request

parthea pushed a commit that referenced this pull request

parthea pushed a commit that referenced this pull request

* feat: bulk writer 555 rate_limiter (#368)

* added 555 throttle utility

* Update google/cloud/firestore_v1/throttle.py

Co-authored-by: Tres Seaver <tseaver@palladion.com>

* added ability to request a number of tokens

* replaced Callable now parameter with module function

* updated tests

* renamed throttle -> ramp up

* improved docstrings

* linting

* fixed test coverage

* rename to RateLimiter and defer clock to first op

* linting

Co-authored-by: Tres Seaver <tseaver@palladion.com>

* feat: added new batch class for BulkWriter (#397)

* feat: added new batch class for BulkWriter

* updated docstring to use less colloquial language

* feat: BulkWriter implementation (#384)

* feat: added `write` method to batch classes

* added docstrings to all 3 batch classes

instead of just the base

* updated batch classes to remove control flag

now branches logic via subclasses

* fixed broken tests off abstract class

* fixed docstring

* refactored BulkWriteBatch

this commit increases the distance between WriteBatch and BulkWriteBatch

* began adding [Async]BulkWriter

* continued implementation

* working impl or BW

* tidied up BW impl

* beginning of unit tests for BW

* fixed merge problem

* initial set of BW unit tests

* refactored bulkwriter sending mechanism

now consumes off the queue and schedules on the main thread, only going async to actually send

* final CI touch ups

* 🦉 Updates from OwlBot

See https://github.com/googleapis/repo-automation-bots/blob/master/packages/owl-bot/README.md

* 🦉 Updates from OwlBot

See https://github.com/googleapis/repo-automation-bots/blob/master/packages/owl-bot/README.md

* moved BulkWriter parameters to options format

* rebased off master

* test fixes

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

* feat: add retry support for BulkWriter errors (#413)

* parent 0176cc7fef8752433b5c2496046d3a56557eb824
author Craig Labenz <craig.labenz@gmail.com> 1623693904 -0700
committer Craig Labenz <craig.labenz@gmail.com> 1628617523 -0400

feat: add retries to bulk-writer

* fixed rebase error

Co-authored-by: Tres Seaver <tseaver@palladion.com>
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

parthea pushed a commit that referenced this pull request

parthea pushed a commit that referenced this pull request

parthea pushed a commit that referenced this pull request

parthea pushed a commit that referenced this pull request

parthea pushed a commit that referenced this pull request

Fixes #413.

This PR makes sure that the ACK deadline set for the received messages is always consistent with what the leaser uses internally when extending the ACK deadlines for the leased messages.

See the issue description and a [comment](googleapis/python-pubsub#413 (comment)) explaining a possible sequence of events that lead to a bug.

**PR checklist**
- [x] Make sure to open an issue as a [bug/issue](https://github.com/googleapis/python-pubsub/issues/new/choose) before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
- [x] Ensure the tests and linter pass
- [x] Code coverage does not decrease (if any source code was changed)
- [x] Appropriate docs were updated (if necessary)

parthea pushed a commit that referenced this pull request

Ensures that each worker creates a unique Spanner instance
which will be DELETED as the worker is exiting!

Potentially solves the issue with very low administrative
quotas for Cloud Spanner, since now each worker will have
its own limits instead of trying to use the exact same instance.

If this works, we'll be able to massively parallelize our tests!

Also while here, made check `continuous` run ONLY a single job
which will take a couple of hours but worthwhile to avoid quota
problems.

Fixes #413.

parthea pushed a commit that referenced this pull request

* deps: update all deps

* chore: update tests

* chore: run code formatter

* chore: ignore warning

* chore: make script work on Python 3.8

* chore: install setuptools

* deps: update pip before running nox

* deps: upgrade setuptools

* build: remove kokoro presubmit workflows

* build: add migration 1310 test runner

* build: only run units + 1.4 compliance

* build: try this config

* build: only run unit tests on Kokoro

parthea pushed a commit that referenced this pull request

Source-Link: googleapis/synthtool@53ea389
Post-Processor: gcr.io/repo-automation-bots/owlbot-python:latest@sha256:e1793a23ae0ee9aafb2e3a53b564a351f74790dbe3c2d75f8fc3b8c43e5c036c

parthea pushed a commit that referenced this pull request