Fix problem described in CORE-1060 and propose possible patch for CORE-858 #23

wants to merge 2 commits into


None yet
2 participants

mrego commented Jan 23, 2012

Problem described in CORE-1060 was added with the patch in CORE-858.

I'm reverting that patch and sending a new proposal that fix the issue I'm suffering with MySQL described in CORE-858

mrego added some commits Jan 23, 2012

@mrego mrego CORE-1060
Regresion - defaultNullValue in addNotNullConstraint is interpreting the value as a String instead of an Int

Revert "CORE-858"

This reverts commit 8dbe0f0.
@mrego mrego CORE-858
addNotNullConstraint not working in MySQL with boolean values

nvoxland commented Mar 9, 2012

With some of the changes I committed for 3.0 around type management, we may be able to fix this in general without needing to hard-code in the boolean checks. I'll need to do some testing first, though, to try to replicate the problem on 3.0

mrego commented Mar 27, 2012

Once 3.0 is released (or at least an alpha or beta is ready) I can test by myself this issue too. Thanks ;-)

mrego closed this Mar 27, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment