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

fix: throw an error message when setting is_row_processor=True to read a multi param function#1160

Merged
jialuoo merged 9 commits intomainfrom
row-processor
Dec 12, 2024
Merged

fix: throw an error message when setting is_row_processor=True to read a multi param function#1160
jialuoo merged 9 commits intomainfrom
row-processor

Conversation

@jialuoo
Copy link
Copy Markdown
Contributor

@jialuoo jialuoo commented Nov 19, 2024

b/372784945.

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:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

@jialuoo jialuoo requested a review from shobsi November 19, 2024 23:39
@jialuoo jialuoo self-assigned this Nov 19, 2024
@jialuoo jialuoo requested review from a team November 19, 2024 23:39
@product-auto-label product-auto-label bot added size: m Pull request size is medium. api: bigquery Issues related to the googleapis/python-bigquery-dataframes API. labels Nov 19, 2024
Comment thread bigframes/functions/remote_function.py
Comment thread bigframes/session/__init__.py Outdated
Comment thread bigframes/session/__init__.py Outdated
Comment thread bigframes/session/__init__.py Outdated
Comment thread bigframes/session/__init__.py Outdated
Comment thread tests/system/small/test_remote_function.py
Comment thread tests/system/small/test_remote_function.py Outdated
Comment thread tests/system/small/test_remote_function.py Outdated
Comment thread tests/system/small/test_remote_function.py Outdated
Comment thread tests/system/small/test_remote_function.py Outdated
Comment thread bigframes/session/__init__.py Outdated
Comment thread bigframes/session/__init__.py Outdated
Comment thread tests/system/small/test_remote_function.py Outdated
Comment thread tests/system/small/test_remote_function.py Outdated
Comment thread tests/system/small/test_remote_function.py Outdated
Comment thread tests/system/small/test_remote_function.py Outdated
Comment thread bigframes/session/__init__.py Outdated
@jialuoo jialuoo merged commit b2816a5 into main Dec 12, 2024
@jialuoo jialuoo deleted the row-processor branch December 12, 2024 09:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: bigquery Issues related to the googleapis/python-bigquery-dataframes API. size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants