Skip to content

Adding HappyBase Table.counter_set(). - #1541

Merged
dhermes merged 1 commit into
googleapis:masterfrom
dhermes:happybase-counter-set
Feb 29, 2016
Merged

Adding HappyBase Table.counter_set().#1541
dhermes merged 1 commit into
googleapis:masterfrom
dhermes:happybase-counter-set

Conversation

@dhermes

Copy link
Copy Markdown
Contributor

I took the note from HappyBase

Note that application code should never store a incremented or decremented counter value directly; use the atomic Table.counter_inc and Table.counter_dec methods for that.

one step further and just made this NotImplemented.

@dhermesdhermes added the api: bigtable Issues related to the Bigtable API. label Feb 26, 2016
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 26, 2016
@theacodes

Copy link
Copy Markdown
Contributor

Fair enough. LGTM.

@dhermesdhermes assigned jgeewax and unassigned theacodesFeb 26, 2016
@dhermes

Copy link
Copy Markdown
ContributorAuthor

Assigning to @jgeewax to see what he thinks about this.

@jgeewax

Copy link
Copy Markdown
Contributor

This is probably a safe move, and likely "the right thing" to do, however it may break some of the compatibility... which is scary...

Might be better to raise a warning...?

/cc @maxluebbe might want to chime in (can we kill counter_set) ?

@tseaver

Copy link
Copy Markdown
Contributor

LGTM.

@dhermes

Copy link
Copy Markdown
ContributorAuthor

Filed #1549 to resolve question.

dhermes added a commit that referenced this pull request Feb 29, 2016
@dhermes
dhermes merged commit 2ab8edc into googleapis:masterFeb 29, 2016
@dhermes
dhermes deleted the happybase-counter-set branch February 29, 2016 19:51
@theacodestheacodes mentioned this pull request Mar 4, 2016
parthea pushed a commit that referenced this pull request Sep 22, 2023
* Add samples for Cloud Tasks [(#1068)](GoogleCloudPlatform/python-docs-samples#1068)
* Add samples for Cloud Tasks
* Respond to tasks sample review
* Update app engine queues samples
* Address review feedback
* Address review issues and convert pull queue sample to not use API key auth
* Reform pull queues to match appengine queues changes to auth, command line input, readme
* flake8 and fix comment
* Fix Tasks sample test issues.
* Remove queue.yaml, now unused
* Add required flag on certain arguments to command-line tools
* Auto-update dependencies. [(#1116)](GoogleCloudPlatform/python-docs-samples#1116)
* Auto-update dependencies. [(#1133)](GoogleCloudPlatform/python-docs-samples#1133)
* Auto-update dependencies.
* Fix missing http library
Change-Id: I99faa600f2f3f1f50f57694fc9835d7f35bda250
* Update Task Queue samples for beta
* Use full import to get client
* Auto-update dependencies. [(#1186)](GoogleCloudPlatform/python-docs-samples#1186)
* Auto-update dependencies. [(#1217)](GoogleCloudPlatform/python-docs-samples#1217)
* Added "Open in Cloud Shell" buttons to README files [(#1254)](GoogleCloudPlatform/python-docs-samples#1254)
* Add comments and region tags to Cloud Tasks samples [(#1271)](GoogleCloudPlatform/python-docs-samples#1271)
* Standardize on CamelCase, reword confusing endpoint name [(#1288)](GoogleCloudPlatform/python-docs-samples#1288)
* Auto-update dependencies. [(#1309)](GoogleCloudPlatform/python-docs-samples#1309)
* Rename pull to lease and fix name/parent confusion [(#1311)](GoogleCloudPlatform/python-docs-samples#1311)
* Updated region tags for pull queue samples [(#1329)](GoogleCloudPlatform/python-docs-samples#1329)
* Auto-update dependencies. [(#1320)](GoogleCloudPlatform/python-docs-samples#1320)
* Auto-update dependencies. [(#1355)](GoogleCloudPlatform/python-docs-samples#1355)
* Auto-update dependencies. [(#1359)](GoogleCloudPlatform/python-docs-samples#1359)
* Auto-update dependencies.
* Update Cloud Tasks Samples [(#1529)](GoogleCloudPlatform/python-docs-samples#1529)
* passing create task
* Passing tests
* updates to region tags
* update region tags [(#1532)](GoogleCloudPlatform/python-docs-samples#1532)
* update Tasks Sample for App Engine [(#1541)](GoogleCloudPlatform/python-docs-samples#1541)
* update gcloud command for creating queues
* deploys and runs
* update license
* passing tests
* Fix run command [(#1563)](GoogleCloudPlatform/python-docs-samples#1563)
* Updated library for TTS GA [(#1552)](GoogleCloudPlatform/python-docs-samples#1552)
* update gcloud command for Cloud Tasks [(#1566)](GoogleCloudPlatform/python-docs-samples#1566)
* update gcloud command
* update pull queue command
* update pull queue command
* Update Cloud Tasks Push Queue Sample [(#1698)](GoogleCloudPlatform/python-docs-samples#1698)
* deleted pull queues
* updated samples
* fix dependency versions
* [Cloud Tasks] Move samples to new folder [(#2114)](GoogleCloudPlatform/python-docs-samples#2114)
* Move samples to keep consistent with other langauges
* Ad system tests as well
* [Cloud Tasks] Add task with authentication sample [(#2113)](GoogleCloudPlatform/python-docs-samples#2113)
* Add task with authentication sample
* Fix linting
* Fix linting
* Fix spacing
* Update tests with service account
* Move samples and update READMEs
* Update version and linting
* Update task sample comments [(#2156)](GoogleCloudPlatform/python-docs-samples#2156)
* Update task comments
* Update readme
* Update queue name
* update gcloud [(#2208)](GoogleCloudPlatform/python-docs-samples#2208)
* Add protobuf dep and install instructions [(#2250)](GoogleCloudPlatform/python-docs-samples#2250)
* Add Migration Guide Snippets for Cloud Tasks [(#2316)](GoogleCloudPlatform/python-docs-samples#2316)
* Migration guide
* remove app and update migraitonn
* snippets for migration guide - tests added
* lint
* remove print statements
* Styling changes
* Travis trigger
* Update create_http_task.py [(#2187)](GoogleCloudPlatform/python-docs-samples#2187)
Updates `create_http_task.py` to have missing `in_seconds` variable
* Adds updates for samples profiler ... vision [(#2439)](GoogleCloudPlatform/python-docs-samples#2439)
* Update Cloud Tasks library version [(#2516)](GoogleCloudPlatform/python-docs-samples#2516)
* Update to new library
* update library version
* Adds Task name while creating Task [(#2543)](GoogleCloudPlatform/python-docs-samples#2543)
* Task Name in Creating HTTP Task with Token [(#2700)](GoogleCloudPlatform/python-docs-samples#2700)
* chore(deps): update dependency google-cloud-tasks to v1.5.0 [(#3168)](GoogleCloudPlatform/python-docs-samples#3168)
* chore(deps): update dependency googleapis-common-protos to v1.51.0 [(#3171)](GoogleCloudPlatform/python-docs-samples#3171)
This PR contains the following updates:
| Package | Update | Change |
|---|---|---|
| [googleapis-common-protos](https://togithub.com/googleapis/googleapis) | minor | `==1.6.0` -> `==1.51.0` |
---
### Renovate configuration
:date: **Schedule**: At any time (no schedule defined).
:vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied.
:recycle: **Rebasing**: Never, or you tick the rebase/retry checkbox.
:no_bell: **Ignore**: Close this PR and you won't be reminded about this update again.
---
- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box
---
This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#GoogleCloudPlatform/python-docs-samples).
* Simplify noxfile setup. [(#2806)](GoogleCloudPlatform/python-docs-samples#2806)
* chore(deps): update dependency requests to v2.23.0
* Simplify noxfile and add version control.
* Configure appengine/standard to only test Python 2.7.
* Update Kokokro configs to match noxfile.
* Add requirements-test to each folder.
* Remove Py2 versions from everything execept appengine/standard.
* Remove conftest.py.
* Remove appengine/standard/conftest.py
* Remove 'no-sucess-flaky-report' from pytest.ini.
* Add GAE SDK back to appengine/standard tests.
* Fix typo.
* Roll pytest to python 2 version.
* Add a bunch of testing requirements.
* Remove typo.
* Add appengine lib directory back in.
* Add some additional requirements.
* Fix issue with flake8 args.
* Even more requirements.
* Readd appengine conftest.py.
* Add a few more requirements.
* Even more Appengine requirements.
* Add webtest for appengine/standard/mailgun.
* Add some additional requirements.
* Add workaround for issue with mailjet-rest.
* Add responses for appengine/standard/mailjet.
Co-authored-by: Renovate Bot <bot@renovateapp.com>
* Remove dependency googleapis-common-protos [(#3955)](GoogleCloudPlatform/python-docs-samples#3955)
* Update dependency googleapis-common-protos to v1.52.0
* Update requirements.txt
* Update requirements.txt
* Update requirements.txt
Co-authored-by: Bu Sun Kim <8822365+busunkim96@users.noreply.github.com>
* Replace GCLOUD_PROJECT with GOOGLE_CLOUD_PROJECT. [(#4022)](GoogleCloudPlatform/python-docs-samples#4022)
* [tasks] testing: use fixtures for the queue [(#4049)](GoogleCloudPlatform/python-docs-samples#4049)
fixes#4045fixes#4044
I don't know why these tests started to fail, but anyways we'd better
use fixtures and temporary queues.
* add python snippets and tests for creating, listing, and deleting queues [(#4012)](GoogleCloudPlatform/python-docs-samples#4012)
* add python snippets and tests for creating, listing, and deleting queues
* fix grammar
* update licenses
* apply suggested fixes and format with black
* refine delete_queue_test with fixture for setup
* utilize fixtures and match format of create_http_task_test
* utilize fixtures in list_queues_test and create_queue_test
* make create_queue_test call the right function
* still attempt to delete queue after test runs in case of failure
* attempt to delete queue in case of failure, using try/except approach
* add print when NotFound is caught
* fix import
Co-authored-by: Averi Kitsch <akitsch@google.com>
Co-authored-by: Takashi Matsuo <tmatsuo@google.com>
* docs(tasks): service_account_email parameter example [(#4183)](GoogleCloudPlatform/python-docs-samples#4183)
* docs(tasks): serviceAccountEmail parameter example
* docs(tasks): parameter name camel => snake case
* docs(tasks): comment order = function param order
* chore(deps): update dependency pytest to v5.4.3 [(#4279)](GoogleCloudPlatform/python-docs-samples#4279)
* chore(deps): update dependency pytest to v5.4.3
* specify pytest for python 2 in appengine
Co-authored-by: Leah Cole <coleleah@google.com>
* Update dependency pytest to v6 [(#4390)](GoogleCloudPlatform/python-docs-samples#4390)
* tasks: added json content-type request [(#4473)](GoogleCloudPlatform/python-docs-samples#4473)
- added json payload compatibility
- fix imports and code block used on https://cloud.google.com/tasks/docs/creating-http-target-tasks#python
## Description
Fixes #<ISSUE-NUMBER>
Note: It's a good idea to open an issue first for discussion.
## Checklist
- [ ] I have followed [Sample Guidelines from AUTHORING_GUIDE.MD](https://github.com/GoogleCloudPlatform/python-docs-samples/blob/master/AUTHORING_GUIDE.md)
- [ ] README is updated to include [all relevant information](https://github.com/GoogleCloudPlatform/python-docs-samples/blob/master/AUTHORING_GUIDE.md#readme-file)
- [ ] **Tests** pass: `nox -s py-3.6` (see [Test Environment Setup](https://github.com/GoogleCloudPlatform/python-docs-samples/blob/master/AUTHORING_GUIDE.md#test-environment-setup))
- [ ] **Lint** pass: `nox -s lint` (see [Test Environment Setup](https://github.com/GoogleCloudPlatform/python-docs-samples/blob/master/AUTHORING_GUIDE.md#test-environment-setup))
- [ ] These samples need a new **API enabled** in testing projects to pass (let us know which ones)
- [ ] These samples need a new/updated **env vars** in testing projects set to pass (let us know which ones)
- [ ] Please **merge** this PR for me once it is approved.
- [ ] This sample adds a new sample directory, and I updated the [CODEOWNERS file](https://github.com/GoogleCloudPlatform/python-docs-samples/blob/master/.github/CODEOWNERS) with the codeowners for this sample
* docs: add samples from python-docs-samples/tasks
* chore: update templates
Co-authored-by: Andrew Gorcester <andrew.gorcester@gmail.com>
Co-authored-by: Andrew Gorcester <gorcester@google.com>
Co-authored-by: DPE bot <dpebot@google.com>
Co-authored-by: michaelawyu <chenyumic@google.com>
Co-authored-by: ellenevans <35748459+ellenevans@users.noreply.github.com>
Co-authored-by: Averi Kitsch <akitsch@google.com>
Co-authored-by: michaelawyu <michael.a.w.yu@hotmail.com>
Co-authored-by: Noah Negrey <nnegrey@users.noreply.github.com>
Co-authored-by: Alex Voorhees <alex@cloudbakers.com>
Co-authored-by: Gus Class <gguuss@gmail.com>
Co-authored-by: Sarath Kaul <kaul.sarath@gmail.com>
Co-authored-by: Sarath Kaul <sarath.kaul@searce.com>
Co-authored-by: WhiteSource Renovate <bot@renovateapp.com>
Co-authored-by: Kurtis Van Gent <31518063+kurtisvg@users.noreply.github.com>
Co-authored-by: Bu Sun Kim <8822365+busunkim96@users.noreply.github.com>
Co-authored-by: Takashi Matsuo <tmatsuo@google.com>
Co-authored-by: Aaron Johnson <aaronmjohnson29@gmail.com>
Co-authored-by: Adam Ross <adamross@google.com>
Co-authored-by: Leah Cole <coleleah@google.com>
Co-authored-by: Joab Leite S. Neto <leitejoab@gmail.com>
parthea added a commit that referenced this pull request Nov 24, 2025
parthea pushed a commit that referenced this pull request Nov 26, 2025
* feat: adds support for X509 workload credential type
* fix: PR comments
* Apply suggestions from code review
Co-authored-by: Leo <39062083+lsirac@users.noreply.github.com>
* fix: responding to PR comments
* Apply suggestions from code review
Co-authored-by: Carl Lundin <108372512+clundin25@users.noreply.github.com>
* renaming functions, adding comments, and removing auth_request temp var
* Apply suggestions from code review
Co-authored-by: Carl Lundin <108372512+clundin25@users.noreply.github.com>
* chore: Update test credentials.
* linting
---------
Co-authored-by: Leo <39062083+lsirac@users.noreply.github.com>
Co-authored-by: Carl Lundin <108372512+clundin25@users.noreply.github.com>
Co-authored-by: Carl Lundin <clundin@google.com>
parthea added a commit that referenced this pull request Mar 6, 2026
#1541)
* bug: fixes discrepancy btwn python-api-core & bigquery re object default timeout
* Fix: loosen ipywidget dependency (#1504)
* fix: updates ipywidget dependency
* fix: updates ipywidget version number
* chore(main): release 3.6.0 (#1490)
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
* docs: Remove < 3.11 reference from README (#1502)
* chore(python): upgrade gcp-releasetool in .kokoro [autoapprove] (#1508)
Source-Link: googleapis/synthtool@5f2a608
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:8555f0e37e6261408f792bfd6635102d2da5ad73f8f09bcb24f25e6afb5fac97
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
* feat: add `connection_properties` and `create_session` to `LoadJobConfig` (#1509)
* feat: added `connection_properties` and `create_session` in load job
* chore(deps): update all dependencies (#1501)
* chore(deps): update all dependencies
* 🦉 Updates from OwlBot post-processor
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
---------
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
* chore(deps): update all dependencies (#1513)
* feat: add default_query_job_config property and property setter to BQ client (#1511)
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
- [x] Make sure to open an issue as a [feature request](https://togithub.com/googleapis/python-bigquery/issues/1512) 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)
Fixes - [feature request](https://togithub.com/googleapis/python-bigquery/issues/1512)🦕
- [internal bug](https://b.corp.google.com/issues/271044948)
* chore(deps): update all dependencies (#1514)
* chore(deps): update dependency charset-normalizer to v3.1.0 (#1518)
* chore(main): release 3.7.0 (#1507)
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
* feat: expose configuration property on CopyJob, ExtractJob, LoadJob, QueryJob (#1521)
* feat: expose configuration property on CopyJob, ExtractJob, LoadJob, QueryJob
Note for google-cloud-bigquery developers: This also refactors these classes so
that `_set_properties` does not modify the `_properties` dictionary in-place.
Doing so was also mutating the request object, making it difficult to debug
what request was _actually_ sent. Before this change, many tests hallucinated
that the request was always equal to the response.
* E google.api_core.exceptions.BadRequest: 400 Clone operation with write disposition WRITE_TRUNCATE is not supported. Please try again with WRITE_EMPTY.
* chore(deps): Update nox in .kokoro/requirements.in [autoapprove] (#1527)
Source-Link: googleapis/synthtool@92006bb
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:2e247c7bf5154df7f98cce087a20ca7605e236340c7d6d1a14447e5c06791bd6
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
* feat: add default LoadJobConfig to Client (#1526)
* feat: add bool, int, float, string dtype to to_dataframe (#1529)
* fix: loosen ipywidgets restrictions further to address ipython compatibility issues (#1531)
* fix: loosen ipywidgets restrictions further to address ipython compatibility issues
* include ipywidgets in prerelease deps
* show all package versions
* add ipykernel dependency
* ipykernel in noxfile
* oops
* chore(main): release 3.8.0 (#1525)
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
* fix: keyerror when the load_table_from_dataframe accesses a unmapped dtype dataframe index (#1535)
* feat: expose query job on dbapi cursor (#1520)
Co-authored-by: Tim Swast <swast@google.com>
* chore(main): release 3.9.0 (#1537)
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
* chore: update tests to be compatible with pandas 2.0 (#1538)
* chore: update tests to be compatible with pandas 2.0
* use StringDtype without storage argument
* avoid Float64Dtype on older pandas
* chore(deps): update all dependencies (#1522)
* chore: updates minimum version of bqstorage (#1542)
* chore: updates minimum version of bqstorage
* removes unneeded test
* updates linting, removes unneeded comment
* updates conditional checks, comments, adds test
* Removes test, adds pragma no cover
* Removes test
* fix linting error
---------
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Atsushi Yamamoto <yamaatsushi927@gmail.com>
Co-authored-by: gcf-owl-bot[bot] <78513119+gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Shobhit Singh <just.shobhit@gmail.com>
Co-authored-by: Mend Renovate <bot@renovateapp.com>
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
Co-authored-by: chelsea-lin <124939984+chelsea-lin@users.noreply.github.com>
Co-authored-by: Tim Swast <swast@google.com>
Co-authored-by: r1b <robert.cole.jensen@gmail.com>
parthea pushed a commit that referenced this pull request Mar 9, 2026
fixes#1541 : Send entire object checksums via upload methods and remove it from Object properties.
* In recent release, v3.3.1 this library provided an option for users to specify entire object checksum before performing resumable upload. It introduced the bug #1541 * This PR is fixing it by providing the option to specify entire object checksum via upload methods - `upload_from_string`, `upload_from_file` and `upload_from_filename`. * **Note**: Ability to specify entire upload checksum is applicable only for [Resumable Uploads](https://cloud.google.com/storage/docs/performing-resumable-uploads) , which in this library is applicable for uploads of object greater than 8 MiB
parthea pushed a commit that referenced this pull request Apr 1, 2026
* feat!: set `allow_large_results=False` by default to optimize small result operations
* adjust progress bar test for large result
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigtableIssues related to the Bigtable API.cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@dhermes@theacodes@jgeewax@tseaver@googlebot