Skip to content

Add materializationDataset as optional#1376

Merged
davidrabinowitz merged 7 commits intoGoogleCloudDataproc:masterfrom
davidrabinowitz:matview
Oct 3, 2025
Merged

Add materializationDataset as optional#1376
davidrabinowitz merged 7 commits intoGoogleCloudDataproc:masterfrom
davidrabinowitz:matview

Conversation

@davidrabinowitz
Copy link
Member

No description provided.

@davidrabinowitz
Copy link
Member Author

/gcbrun

@davidrabinowitz
Copy link
Member Author

/gcbrun

@davidrabinowitz
Copy link
Member Author

/gcbrun

@davidrabinowitz
Copy link
Member Author

/gcbrun

@kristopherkane
Copy link
Contributor

kristopherkane commented Sep 19, 2025

Thanks for this David. Getting this into a release that resolves #1398 would be really helpful for us. Although, I'm not sure how this PR plays in with https://github.com/GoogleCloudDataproc/spark-bigquery-connector/pull/1345/files which seems really nice too. If we could avoid the 'too large' problem and also not have to provide the materialization dataset that would be a big win.

@davidrabinowitz
Copy link
Member Author

Hi @kristopherkane

Thanks for this David. Getting this into a release that resolves #1398 would be really helpful for us. Although, I'm not sure how this PR plays in with https://github.com/GoogleCloudDataproc/spark-bigquery-connector/pull/1345/files which seems really nice too. If we could avoid the 'too large' problem and also not have to provide the materialization dataset that would be a big win.

Thanks, the idea is to make materializationDataset optional for users who need it (due to permissions issues, for example), but most users would be able to avoid it.

@davidrabinowitz
Copy link
Member Author

/gcbrun

1 similar comment
@davidrabinowitz
Copy link
Member Author

/gcbrun

@davidrabinowitz
Copy link
Member Author

/gcbrun

@davidrabinowitz davidrabinowitz merged commit 5e6c300 into GoogleCloudDataproc:master Oct 3, 2025
7 checks passed
@davidrabinowitz davidrabinowitz deleted the matview branch October 3, 2025 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants