Skip to content

bigtable: support local emulator#1567

Merged
callmehiphop merged 3 commits intogoogleapis:masterfrom
stephenplusplus:spp--1506
Oct 4, 2016
Merged

bigtable: support local emulator#1567
callmehiphop merged 3 commits intogoogleapis:masterfrom
stephenplusplus:spp--1506

Conversation

@stephenplusplus
Copy link
Contributor

@stephenplusplus stephenplusplus commented Sep 6, 2016 •
edited
Loading

Fixes #1506

To Dos

  • Overview docs showing how to connect with the emulator

@stephenplusplus stephenplusplus added enhancement api: bigtable Issues related to the Bigtable API. labels Sep 6, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 6, 2016
@coveralls
Copy link

coveralls commented Sep 6, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling 1cc8dba on stephenplusplus:spp--1506 into cb71873 on GoogleCloudPlatform:master.

@googlebot
Copy link

googlebot commented Sep 19, 2016

We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm.

@googlebot googlebot added cla: no This human has *not* signed the Contributor License Agreement. and removed cla: yes This human has signed the Contributor License Agreement. labels Sep 19, 2016
@callmehiphop
Copy link
Contributor

callmehiphop commented Sep 19, 2016

@stephenplusplus I've added a commit, I think the only thing that was missing was making sure that all the service (admin/instance/etc.) were also updated. Obviously the instance API doesn't work with the mentioned emulator, but parts of the table admin api do.

@arbesfeld any chance you'd want to give this a test drive?

@coveralls
Copy link

coveralls commented Sep 20, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling f741be2 on stephenplusplus:spp--1506 into 35573c3 on GoogleCloudPlatform:master.

@garye
Copy link

garye commented Oct 4, 2016

@callmehiphop What's the status of this? Is it something we can merge or does it need more testing?

@callmehiphop callmehiphop merged commit f78bbdb into googleapis:master Oct 4, 2016
@callmehiphop
Copy link
Contributor

callmehiphop commented Oct 4, 2016

@garye should be good to go!

@coveralls
Copy link

coveralls commented Oct 4, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling d4f3d58 on stephenplusplus:spp--1506 into e2c456f on GoogleCloudPlatform:master.

sofisl pushed a commit that referenced this pull request Feb 24, 2026
@fhinkel
chore: use cloud-rad publication process (#1567)
ba40e47
sofisl pushed a commit that referenced this pull request Feb 25, 2026
@shaffeeullah @gcf-owl-bot @ddelgrosso1
test: conformance testing for retries (#1566)
9757193 
* feat: added library methods for conformance testing (#1553)

* feat: added library methods for conformance testing

* 🦉 Updates from OwlBot

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

* fixed header

* responded to PR comment, addressed some TODOs

* resolved TODO

* removed functions that dont make requests

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

* feat: add method mapping json file (#1569)

* feat: add method mapping json file

* add deleteResumableCache to mapping

* WIP: retry strategy runner code (#1567)

* feat: retry strategy runner code

* fixed license

* removed unused option

* 🦉 Updates from OwlBot

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

* built out tests further, still doesnt work

* 🦉 Updates from OwlBot

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

* built out more functionality

* 🦉 Updates from OwlBot

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

* began refactor

* 🦉 Updates from OwlBot

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

* moved functions

* refactored code

* 🦉 Updates from OwlBot

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

* Update libraryMethods.ts

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

* fix: fixed runtime errors. all the tests still fail, but they run now (#1591)

* fix: fixed runtime errors. all the tests still fail, but they run now

* 🦉 Updates from OwlBot

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

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

* fix: exported functions

* 🦉 Updates from OwlBot

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

* test: pull testbench docker image, kokoro presubmit conformance test … (#1590)

* test: pull testbench docker image, kokoro presubmit conformance test hooks

* use async function instead of explicit promise return

* move file to node12 directory

* fix file naming and linter issues

* fix: retry strategy bug fixes (#1594)

* bug fixes

* removed logs

* 🦉 Updates from OwlBot

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

* some tests pass

* fixed some more tests

* saves file

* 🦉 Updates from OwlBot

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

* uncommented functions

* 🦉 Updates from OwlBot

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

* removed duplicate storage definition

* changed precondition options

* down to 7 failing tests!

* 🦉 Updates from OwlBot

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

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

* feat: check that instructions were utilized in testbench (#1601)

* feat: check that instructions were utilized in testbench

* feedback

* fix: remove all files from bucket before delete (#1603)

* feat: move test creation and header set to beforeEach (#1605)

* feat: move test creation and header set to beforeEach

* 🦉 Updates from OwlBot

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

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

* fix: create client for each test instead of globally (#1608)

* fix: remove bucketGet (#1610)

* fix: return promise from stream functions and resolve on end (#1612)

* fix: move createReadStream to correct mapping location (#1613)

* fix: remap bucket set storage class and bucket set metadata (#1615)

* fix: remap bucket set storage class

* fix: remap bucketSetMetadata

* fix: fixed bucketUploadMultipart (#1627)

* fix: fixed bucketUploadMultipart

* 🦉 Updates from OwlBot

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

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

* fix: conformance test fixes for deleteFiles, fileDelete, and copy (#1636)

* fix: add copy to invocation map

* fix: fix deleteFiles and fileDelete conformance tests

* linter fixes

* fix: only send metageneration to bucket related tests (#1641)

* fix: remove tests for createWriteStream as the function is not retryable

* fix: only send metageneration to bucket related tests

* linter

* fix: set retention period before file creation (#1651)

* set retention period before file creation

* removed log statement

* removed log statement

* fix: set correct bucket precondition (#1653)

* fix: set correct bucket precondition

* Update retryStrategy.ts

* 🦉 Updates from OwlBot

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

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

* fix: combine conformance test (#1656)

* fix: update test bucket (#1657)

* fix: combine conformance test

* fix: only return bucket with preconditions if the test says so

* 🦉 Updates from OwlBot

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

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

* fix: fixed storage.objects.patch tests (#1658)

* fix: fixed storage.objects.patch tests

* 🦉 Updates from OwlBot

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

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

* fix: refactor tests (#1659)

* fix: refactor tests

* 🦉 Updates from OwlBot

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

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

* chore: build headers for isPublic from interceptors (#1665)

* chore: build headers for isPublic from interceptors

* simplify code

* tests: removed failing tests (internal issue b/203793664) (#1674)

* fix: add conformance project id (#1673)

* fix: add conformance test project id settting

* lint

* increase timeout for before hook

* specify correct host to docker port mapping

* add stop command for docker container

* chore: use import on json files instead of fs read (#1675)

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Denis DelGrosso <85250797+ddelgrosso1@users.noreply.github.com>
Co-authored-by: Denis DelGrosso <ddelgrosso@google.com>
sofisl pushed a commit that referenced this pull request Feb 26, 2026
@renovate-bot
chore(deps): update dependency @types/concat-stream to v2 (#1567)
643842e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@callmehiphop callmehiphop

Labels

api: bigtable Issues related to the Bigtable API. cla: no This human has *not* signed the Contributor License Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@stephenplusplus @coveralls @googlebot @callmehiphop @garye