Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Document limit pushdown in BigQuery connector #24285

Merged
merged 1 commit into from
Jan 9, 2025

Conversation

mosabua
Copy link
Member

@mosabua mosabua commented Nov 27, 2024

Description

I have no idea if the connector supports others as well, but we have to start somewhere.

Additional context and related issues

Follow up to #24021

Also fyi @jhlodin

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.

docs/src/main/sphinx/connector/bigquery.md Outdated Show resolved Hide resolved
docs/src/main/sphinx/connector/bigquery.md Outdated Show resolved Hide resolved
@mosabua mosabua force-pushed the bq branch 2 times, most recently from cc17711 to 909e35c Compare December 3, 2024 17:50
@mosabua
Copy link
Member Author

mosabua commented Dec 3, 2024

Updated .. its not ideal but .. wdyt @jhlodin @krvikash @ebyhr

Copy link

github-actions bot commented Jan 1, 2025

This pull request has gone a while without any activity. Tagging for triage help: @mosabua

@github-actions github-actions bot added the stale label Jan 1, 2025
@mosabua mosabua changed the title Add limit pushdown to BigQuery docs Document limit pushdown in BigQuery connector Jan 9, 2025
@mosabua
Copy link
Member Author

mosabua commented Jan 9, 2025

I think I addressed the details now @Praveen2112 and @ebyhr

@mosabua
Copy link
Member Author

mosabua commented Jan 9, 2025

Thank you @ebyhr

@mosabua mosabua merged commit 6b868e3 into trinodb:master Jan 9, 2025
8 checks passed
@mosabua mosabua deleted the bq branch January 9, 2025 23:26
@github-actions github-actions bot added this to the 469 milestone Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants