Skip to content

Conversation

@candiduslynx
Copy link
Contributor

@candiduslynx candiduslynx commented Jun 10, 2024

BEGIN_COMMIT_OVERRIDE
fix: Support Apache Arrow types with non-default options (#18198). Previously some Apache Arrow types (namely, timestamp with non-default time zone) would result in plugin failure.

BREAKING-CHANGE: Apache Arrow timestamp now always maps to BigQuery timestamp type (#18198). Previously, timestamp with nanoseconds precision was mapped to a record type.

END_COMMIT_OVERRIDE

Instead of #18126
Fixes https://github.com/cloudquery/cloudquery-issues/issues/1786 (internal issue)

@candiduslynx candiduslynx marked this pull request as ready for review June 10, 2024 16:06
@candiduslynx candiduslynx requested review from a team, erezrokah and murarustefaan and removed request for a team June 10, 2024 16:06
@hermanschaaf hermanschaaf changed the title fix!: Properly support Apache Arrow types with non-default options fix!: Support Apache Arrow types with non-default options Jun 11, 2024
@candiduslynx candiduslynx added the automerge Automatically merge once required checks pass label Jun 11, 2024
@kodiakhq kodiakhq bot merged commit 4b775f9 into main Jun 11, 2024
@kodiakhq kodiakhq bot deleted the fix/bq/arrow branch June 11, 2024 08:17
kodiakhq bot pushed a commit that referenced this pull request Jun 11, 2024
🤖 I have created a release *beep* *boop*
---


## [4.0.0](plugins-destination-bigquery-v3.6.1...plugins-destination-bigquery-v4.0.0) (2024-06-11)


### ⚠ BREAKING CHANGES

* Apache Arrow `timestamp` now always maps to BigQuery `timestamp` type ([#18198](#18198)). Previously, `timestamp` with `nanoseconds` precision was mapped to a `record` type.

### Features

* Remove logging of error events to Sentry in plugins ([#18165](#18165)) ([fc4ff27](fc4ff27))


### Bug Fixes

* **deps:** Update module google.golang.org/api to v0.183.0 ([#18195](#18195)) ([c560cfa](c560cfa))
* Support Apache Arrow types with non-default options ([#18198](#18198)). Previously some Apache Arrow types (namely, `timestamp` with non-default time zone) would result in plugin failure. ([4b775f9](4b775f9))

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

Labels

area/plugin/destination/bigquery automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants