Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Moving to iterators in DNS.#2532

Closed
dhermes wants to merge 1 commit into
googleapis:mastergoogleapis/google-cloud-python:masterfrom
dhermes:dns-iteratorsdhermes/google-cloud-python:dns-iteratorsCopy head branch name to clipboard
Closed

Moving to iterators in DNS.#2532
dhermes wants to merge 1 commit into
googleapis:mastergoogleapis/google-cloud-python:masterfrom
dhermes:dns-iteratorsdhermes/google-cloud-python:dns-iteratorsCopy head branch name to clipboard

Conversation

@dhermes

@dhermes dhermes commented Oct 12, 2016

Copy link
Copy Markdown
Contributor

NOTE: Has #2531 as diffbase.

In doing this, I realized it's a bit tedious to have to write two classes (page and iterator) for each method. It may be more user-friendly to collapse back into one class. I think I could do it mostly by just making anything that is accessed via self.page.foo as self.page_foo.

@dhermes dhermes added the api: dns Issues related to the Cloud DNS API. label Oct 12, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 12, 2016
@dhermes

dhermes commented Oct 17, 2016

Copy link
Copy Markdown
Contributor Author

@daspecster @tseaver @jonparrott PTAL.

There will be about 5 more PRs like this for each package which does page token list_foo().

@theacodes theacodes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks pretty straightforward to me.

@dhermes

dhermes commented Oct 17, 2016

Copy link
Copy Markdown
Contributor Author

TRAVIS!

@theacodes

Copy link
Copy Markdown
Contributor

TRAVIS!

No travis for you, I used up all of his cycles in google-auth. :P

@tseaver

tseaver commented Oct 17, 2016

Copy link
Copy Markdown
Contributor

I don't really care for the inheritance-based iterator approach: would it be better to tackle that first in #2531#2548? Or merge this and then maybe refactor, killing of the subclasses?

@dhermes

dhermes commented Oct 18, 2016

Copy link
Copy Markdown
Contributor Author

@tseaver Can you weigh in on #2558 ASAP?

@dhermes dhermes closed this Oct 18, 2016
@dhermes dhermes deleted the dns-iterators branch October 18, 2016 20:21
ohmayr pushed a commit that referenced this pull request Mar 13, 2026
Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
Co-authored-by: Chalmer Lowe <chalmerlowe@google.com>
chalmerlowe pushed a commit that referenced this pull request Apr 1, 2026
PR created by the Librarian CLI to initialize a release. Merging this PR
will auto trigger a release.

Librarian Version: v0.7.0
Language Image:
us-central1-docker.pkg.dev/cloud-sdk-librarian-prod/images-prod/python-librarian-generator@sha256:160860d189ff1c2f7515638478823712fa5b243e27ccc33a2728669fa1e2ed0c
<details><summary>bigframes: 2.39.0</summary>

##
[2.39.0](googleapis/python-bigquery-dataframes@v2.38.0...v2.39.0)
(2026-03-31)

### Features

* add `df.bigquery.ai.forecast` method to pandas dataframe accessor
(#2518)
([1126cec9](googleapis/python-bigquery-dataframes@1126cec9))

* support full round-trip persistence for multimodal reference cols
(#2511)
([494a0a11](googleapis/python-bigquery-dataframes@494a0a11))

* expose DataFrame.bigquery in both pandas and bigframes DataFrames
(#2533)
([69fe3176](googleapis/python-bigquery-dataframes@69fe3176))

### Bug Fixes

* to_gbq may swap data columns when replace table (#2532)
([17ecc65e](googleapis/python-bigquery-dataframes@17ecc65e))

* handle aggregate operations on empty selections (#2510)
([34fb5daa](googleapis/python-bigquery-dataframes@34fb5daa))

* Localize BigQuery log suppression for gbq.py (#2541)
([af49ca29](googleapis/python-bigquery-dataframes@af49ca29))

* Respect remote function config changes even if logic unchanged (#2512)
([b9524284](googleapis/python-bigquery-dataframes@b9524284))

* support melting empty DataFrames without crashing (#2509)
([e8c4603](googleapis/python-bigquery-dataframes@e8c46032))

### Performance Improvements

* Make executor data uploads async internally (#2529)
([96597f0b](googleapis/python-bigquery-dataframes@96597f0b))

### Documentation

* gemini retouch of the index page for seo (#2514)
([2e5311e2](googleapis/python-bigquery-dataframes@2e5311e2))

* Rename Blob column references to ObjectRef column (#2535)
([44e0ffd9](googleapis/python-bigquery-dataframes@44e0ffd9))

</details>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: dns Issues related to the Cloud DNS API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.