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

Parse External table partitions correctly #2284

Merged
merged 1 commit into from
Dec 21, 2023

Conversation

sfc-gh-jcieslak
Copy link
Collaborator

@sfc-gh-jcieslak sfc-gh-jcieslak commented Dec 20, 2023

Fix for #2283
Reason: The Terraform on a ListType field returns a list of []any, so the Go assertion to []string failed because the underlying type was not known. A helper expandStringList function should be used instead.

Copy link

Integration tests failure for 69dfc61a31e8325f64948b58ab5a9a0d86635343

@sfc-gh-jcieslak sfc-gh-jcieslak merged commit bbad5c6 into main Dec 21, 2023
6 of 9 checks passed
@sfc-gh-jcieslak sfc-gh-jcieslak deleted the fix-partitioned-by-in-external-tables branch December 21, 2023 11:06
sfc-gh-asawicki pushed a commit that referenced this pull request Dec 21, 2023
🤖 I have created a release *beep* *boop*
---


##
[0.82.0](v0.81.0...v0.82.0)
(2023-12-21)


### 🎉 **What's new:**

* add functions to sdk
([#2205](#2205))
([e542b67](e542b67))


### 🔧 **Misc**

* Add short scripts used to fetch all currently opened issues
([#2288](#2288))
([0b5ce4e](0b5ce4e))
* Bump dependencies and fix linter complaints
([#2300](#2300))
([124e862](124e862))


### 🐛 **Bug fixes:**

* Fix resource monitor alter validations in the SDK
([#2298](#2298))
([74d19d0](74d19d0))
* Fix email notification integration
([#2292](#2292))
([70edd3e](70edd3e))
* Fix for #2283 
([#2284](#2284))
([bbad5c6](bbad5c6))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: snowflake-release-please[bot] <105954990+snowflake-release-please[bot]@users.noreply.github.com>
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.

2 participants