[ZEPPELIN-6408] Modernize BigQuery Interpreter with google-cloud-bigquery client#5191
Open
jongyoul wants to merge 4 commits intoapache:masterfrom
Open
[ZEPPELIN-6408] Modernize BigQuery Interpreter with google-cloud-bigquery client#5191jongyoul wants to merge 4 commits intoapache:masterfrom
jongyoul wants to merge 4 commits intoapache:masterfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Updates Zeppelin’s BigQuery interpreter to use the modern google-cloud-bigquery Java client library, replacing the legacy BigQuery API client and documenting the new authentication behavior.
Changes:
- Refactors
BigQueryInterpreterto execute queries and iterate results usinggoogle-cloud-bigquery. - Adds an authentication fallback flow that prompts for a Service Account JSON key when ADC isn’t available.
- Updates BigQuery interpreter docs/README and adjusts the integration test context setup.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 14 comments.
Show a summary per file
| File | Description |
|---|---|
bigquery/src/main/java/org/apache/zeppelin/bigquery/BigQueryInterpreter.java |
Migrates execution/auth/cancel logic to google-cloud-bigquery and adds GUI-based SA JSON fallback. |
bigquery/pom.xml |
Replaces legacy Google API deps with google-cloud-bigquery and updates Guava dependency. |
bigquery/src/test/java/org/apache/zeppelin/bigquery/BigQueryInterpreterTest.java |
Updates resource loading and sets AuthenticationInfo on the context. |
docs/interpreter/bigquery.md |
Updates user-facing docs for new client + authentication modes and refreshes examples. |
bigquery/README.md |
Updates module README with new auth/config/test instructions. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
bigquery/src/main/java/org/apache/zeppelin/bigquery/BigQueryInterpreter.java
Outdated
Show resolved
Hide resolved
bigquery/src/main/java/org/apache/zeppelin/bigquery/BigQueryInterpreter.java
Outdated
Show resolved
Hide resolved
bigquery/src/main/java/org/apache/zeppelin/bigquery/BigQueryInterpreter.java
Outdated
Show resolved
Hide resolved
bigquery/src/main/java/org/apache/zeppelin/bigquery/BigQueryInterpreter.java
Outdated
Show resolved
Hide resolved
bigquery/src/main/java/org/apache/zeppelin/bigquery/BigQueryInterpreter.java
Outdated
Show resolved
Hide resolved
bigquery/src/test/java/org/apache/zeppelin/bigquery/BigQueryInterpreterTest.java
Outdated
Show resolved
Hide resolved
bigquery/src/main/java/org/apache/zeppelin/bigquery/BigQueryInterpreter.java
Show resolved
Hide resolved
bigquery/src/main/java/org/apache/zeppelin/bigquery/BigQueryInterpreter.java
Outdated
Show resolved
Hide resolved
bigquery/src/main/java/org/apache/zeppelin/bigquery/BigQueryInterpreter.java
Show resolved
Hide resolved
…x indentation, and add security warnings
…ob timeout, and cancel error handling
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What is this PR for?
The current BigQuery interpreter uses the legacy
google-api-services-bigquerylibrary, which is outdated and lacks modern features like the Storage Read API. This PR updates the interpreter to use the moderngoogle-cloud-bigqueryCloud Client Library.Key improvements:
google-cloud-bigquery(v2.38.0).What type of PR is it?
Improvement, Refactoring
Todos
bigquery/pom.xmlwith modern Google Cloud dependencies.BigQueryInterpreter.javato use the new client library.z.inputGUI.BigQueryInterpreterTestagainst a live GCP environment.What is the Jira issue?
How should this be tested?
BigQueryInterpreterTestto use the new client. Verified against a live GCP project using public datasets.Screenshots (if appropriate)
(Not applicable for this library update, but functional verification confirmed via logs)
Questions: