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

fix: alter table column data type #2607

Merged
merged 1 commit into from
Mar 11, 2024

Conversation

sfc-gh-jcieslak
Copy link
Collaborator

@sfc-gh-jcieslak sfc-gh-jcieslak commented Mar 8, 2024

Fix for #2588
In the alter column call collate was always added which caused the error non-text (and probably text too) data types

Test Plan

  • test case for data type alter

References

Copy link

github-actions bot commented Mar 8, 2024

Integration tests failure for 95935c4178c12481a967b0c92fb17ea376b39b6c

@sfc-gh-jcieslak sfc-gh-jcieslak merged commit 538b6dc into main Mar 11, 2024
9 of 10 checks passed
@sfc-gh-jcieslak sfc-gh-jcieslak deleted the remove-collate-from-non-text-column-types branch March 11, 2024 11:44
sfc-gh-swinkler pushed a commit that referenced this pull request Mar 19, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.87.3-pre](v0.87.2...v0.87.3-pre)
(2024-03-18)


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

* Add snowflake grant ownership resource
([#2604](#2604))
([bfadd24](bfadd24)),
closes
[#2549](#2549)
[#2199](#2199)
[#2084](#2084)
[#1942](#1942)
[#1875](#1875)


### 🔧 **Misc**

* Fix env variables for tests
([#2603](#2603))
([8bc2437](8bc2437))
* release 0.87.3-pre
([a2be7b9](a2be7b9))


### 🐛 **Bug fixes:**

* alter table column data type
([#2607](#2607))
([538b6dc](538b6dc))
* cgo goreleaser alt solution
([#2613](#2613))
([5d31856](5d31856))

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