Skip to content
This repository was archived by the owner on Mar 6, 2026. It is now read-only.

feat: Expose remote function options (maxBatchingRows, containerCpu, containerMemory) as first class properties - #2305

Closed
chalmerlowe wants to merge 2 commits into
mainfrom
feat-remote-function-options
Closed

feat: Expose remote function options (maxBatchingRows, containerCpu, containerMemory) as first class properties#2305
chalmerlowe wants to merge 2 commits into
mainfrom
feat-remote-function-options

Conversation

@chalmerlowe

@chalmerlowechalmerlowe commented Sep 19, 2025•
edited
Loading

Copy link
Copy Markdown
Collaborator

UPDATE: do not merge. awaiting feedback on the correct location to hang these properties. See convo in #2240

Exposes maxBatchingRows, containerCpu, and containerMemory as first-class properties on the RemoteFunctionOptions class.

Fixes#2240 🦕

@google-labs-jules
feat: Expose more remote functions options as first class properties
7cae208
Exposes `maxBatchingRows`, `containerCpu`, and `containerMemory` as first-class properties on the `RemoteFunctionOptions` class.
Fixes#2240
@chalmerlowe
chalmerlowe requested review from a team and tswastSeptember 19, 2025 18:29
@product-auto-labelproduct-auto-labelBot added the size: m Pull request size is medium. label Sep 19, 2025
@product-auto-labelproduct-auto-labelBot added the api: bigquery Issues related to the googleapis/python-bigquery API. label Sep 19, 2025
@chalmerlowe

chalmerlowe commented Sep 22, 2025

Copy link
Copy Markdown
CollaboratorAuthor

Waiting for a response on the Issue about the intent of the requester. See Issue #2240

@chalmerlowechalmerlowe added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Sep 24, 2025
@chalmerlowechalmerlowe mentioned this pull request Sep 29, 2025
Merged
@chalmerlowe

chalmerlowe commented Sep 29, 2025

Copy link
Copy Markdown
CollaboratorAuthor

OBE. See PR #2311 and Issue #2240 for discussion.

Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@tswasttswastAwaiting requested review from tswast

Assignees

@yirutangyirutang

Labels

api: bigqueryIssues related to the googleapis/python-bigquery API.do not mergeIndicates a pull request not ready for merge, due to either quality or timing.size: mPull request size is medium.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Expose more remote functions options as first class properties

2 participants

@chalmerlowe@yirutang