Skip to content

Conversation

@tseaver
Copy link
Contributor

@tseaver tseaver commented Jun 23, 2015

See #944.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 23, 2015
@dhermes
Copy link
Contributor

dhermes commented Jun 23, 2015

LGTM with emphasis on m == ME

@jgeewax May have a preference here.

@jgeewax
Copy link
Contributor

jgeewax commented Jun 25, 2015

OK so sample code would read something like...

client = datastore.Client(dataset_id='whatever')
client.get(<key here>)

?

If so, then huge +1.

@dhermes
Copy link
Contributor

dhermes commented Jun 25, 2015

@jgeewax Yes

@tseaver Feel free to merge. Do you have most of #944 waiting to be sent out or still working on it?

@tseaver
Copy link
Contributor Author

tseaver commented Jun 25, 2015

It is still in work (I have the ordering / break-up somewhat wrong in #944).

tseaver added a commit that referenced this pull request Jun 25, 2015
Rename 'dataset.Dataset' -> 'client.Client'.
@tseaver tseaver merged commit 119cba3 into googleapis:master Jun 25, 2015
@tseaver tseaver deleted the 944-datastore_client_from_dataset branch June 25, 2015 17:33
dhermes added a commit to dhermes/google-cloud-python that referenced this pull request Jun 25, 2015
@dhermes dhermes mentioned this pull request Jul 10, 2015
parthea pushed a commit that referenced this pull request Aug 21, 2025
* fix: separate create_stream() in pyarrow sample

* remove append_rows()

* lint

* add blank row
This was referenced Sep 12, 2025
parthea pushed a commit that referenced this pull request Sep 16, 2025
* fix: separate create_stream() in pyarrow sample

* remove append_rows()

* lint

* add blank row
parthea pushed a commit that referenced this pull request Sep 18, 2025
parthea added a commit that referenced this pull request Nov 24, 2025
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
parthea pushed a commit that referenced this pull request Nov 24, 2025
* fix: 16-bit hexadecimal formatting for XCTC span IDs

* 🦉 Updates from OwlBot post-processor

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

* addressed nit

* 🦉 Updates from OwlBot post-processor

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

* Fixed test + docstring

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Nov 24, 2025
…y_data (#817) (#946)

* Fix issue 817

* Fix issue 817

* 🦉 Updates from OwlBot post-processor

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

---------

Co-authored-by: venticello <omnisens@gmail.com>
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Jim Morrison <phython@google.com>
parthea pushed a commit that referenced this pull request Nov 25, 2025
* fix: 16-bit hexadecimal formatting for XCTC span IDs

* 🦉 Updates from OwlBot post-processor

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

* addressed nit

* 🦉 Updates from OwlBot post-processor

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

* Fixed test + docstring

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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