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!: EXPOSED-546 [H2] Column with Json default value can not be used as databaseGenerated #2241

Merged
merged 1 commit into from
Sep 19, 2024

Conversation

obabichevjb
Copy link
Collaborator

That bug I found when I was working on disabling sending all the default parameters explicitly in insert statements.

The main reason, before changes the DDL definition of the columns was looking like JSON DEFAULT '{"name":"name","team":"team"}', and it was recognized string, so the new entry had json string instead of json object. After changes DDL looks like DEFAULT JSON '{"name":"name","team":"team"}'


Type of Change

  • Bug fix

Updates/remove existing public API methods:

  • Is breaking change

Affected databases:

  • H2

Related Issues

EXPOSED-546 [H2] Column with Json default value can not be used as databaseGenerated

@obabichevjb obabichevjb force-pushed the obabichev/exposed-546-json-default-value branch 2 times, most recently from 5203316 to a6ef98f Compare September 17, 2024 12:32
@obabichevjb obabichevjb requested a review from joc-a September 17, 2024 13:58
@obabichevjb obabichevjb force-pushed the obabichev/exposed-546-json-default-value branch from a6ef98f to 84b9141 Compare September 18, 2024 07:53
@joc-a joc-a changed the title fix!: EXPOSED-546 [H2] Column with Json default value can not be used as da… fix!: EXPOSED-546 [H2] Column with Json default value can not be used as databaseGenerated Sep 19, 2024
@obabichevjb obabichevjb merged commit 3c0a251 into main Sep 19, 2024
5 checks passed
@obabichevjb obabichevjb deleted the obabichev/exposed-546-json-default-value branch September 19, 2024 13:52
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