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

feat: Add schema comparision to the main request loop for multiplexing to correctly update schema #1865

Merged
GaoleMeng merged 40 commits into
googleapis:mainfrom
GaoleMeng:main
Nov 4, 2022
Merged

feat: Add schema comparision to the main request loop for multiplexing to correctly update schema #1865
GaoleMeng merged 40 commits into
googleapis:mainfrom
GaoleMeng:main

Conversation

@GaoleMeng
Copy link
Copy Markdown
Contributor

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> ☕️

If you write sample code, please follow the samples format.

GaoleMeng and others added 30 commits September 13, 2022 01:58
also fixed a tiny bug inside fake bigquery write impl for getting thre
response from offset
possible the proto schema does not contain this field
@GaoleMeng GaoleMeng requested review from a team and prash-mi November 3, 2022 01:49
@product-auto-label product-auto-label Bot added size: m Pull request size is medium. api: bigquerystorage Issues related to the googleapis/java-bigquerystorage API. labels Nov 3, 2022
@GaoleMeng GaoleMeng requested a review from yirutang November 3, 2022 21:29
@GaoleMeng GaoleMeng changed the title Add schema comparision to the main request loop for multiplexing to correctly update schema feat: Add schema comparision to the main request loop for multiplexing to correctly update schema Nov 3, 2022
Copy link
Copy Markdown
Contributor

@yirutang yirutang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, some nits.

@GaoleMeng GaoleMeng added the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 4, 2022
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 4, 2022
@gcf-owl-bot gcf-owl-bot Bot requested a review from a team November 4, 2022 05:08
@GaoleMeng GaoleMeng merged commit cb18d28 into googleapis:main Nov 4, 2022
gcf-merge-on-green Bot pushed a commit that referenced this pull request Nov 8, 2022
🤖 I have created a release *beep* *boop*
---


## [2.25.0](https://togithub.com/googleapis/java-bigquerystorage/compare/v2.24.2...v2.25.0) (2022-11-08)


### Features

* Add schema comparision to the main request loop for multiplexing to correctly update schema  ([#1865](https://togithub.com/googleapis/java-bigquerystorage/issues/1865)) ([cb18d28](https://togithub.com/googleapis/java-bigquerystorage/commit/cb18d288f78773af60b9fddc583f46a571cbc3f4))


### Documentation

* Remove stale header guidance for AppendRows ([#1866](https://togithub.com/googleapis/java-bigquerystorage/issues/1866)) ([1de23be](https://togithub.com/googleapis/java-bigquerystorage/commit/1de23be7fe84747a4e7ca5b511458a5b378a1170))


### Dependencies

* Update dependency com.google.cloud:google-cloud-bigquery to v2.18.2 ([#1862](https://togithub.com/googleapis/java-bigquerystorage/issues/1862)) ([ca2cafe](https://togithub.com/googleapis/java-bigquerystorage/commit/ca2cafe095eca8b610ec8ca1acf95cf7a561af61))
* Update dependency com.google.cloud:google-cloud-bigquery to v2.19.0 ([#1871](https://togithub.com/googleapis/java-bigquerystorage/issues/1871)) ([4aa967a](https://togithub.com/googleapis/java-bigquerystorage/commit/4aa967a0a30db0037a04b6b47646ae79dfffaead))
* Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.0.6 ([#1870](https://togithub.com/googleapis/java-bigquerystorage/issues/1870)) ([17b2a80](https://togithub.com/googleapis/java-bigquerystorage/commit/17b2a8036cf5706cb753f7116b1fa424a661ba39))
* Update dependency kr.motd.maven:os-maven-plugin to v1.7.1 ([#1868](https://togithub.com/googleapis/java-bigquerystorage/issues/1868)) ([08b8468](https://togithub.com/googleapis/java-bigquerystorage/commit/08b8468dc31fe654f5f7a345377b14f51f19eeea))

---
This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: bigquerystorage Issues related to the googleapis/java-bigquerystorage API. size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants