You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cayenne.apache.org by Aristedes Maniatis <ar...@maniatis.org> on 2011/09/15 10:55:46 UTC
Re: svn commit: r1171003 - in /cayenne/main/trunk: ./ framework/cayenne-jdk1.5-unpublished/src/test/java/org/apache/cayenne/unit/
framework/cayenne-jdk1.5-unpublished/src/test/java/org/apache/cayenne/unit/di/server/
On 15/09/11 6:27 PM, dkazimirchyk@apache.org wrote:
> - -DcayenneTestConnection=${cayenneTestConnection} -Djava.net.preferIPv4Stack=true
> + -DcayenneTestConnection=${cayenneTestConnection} -Djava.net.preferIPv4Stack=true -Dcayenne.runtime.db.collation.assume.ci=${cayenne.runtime.db.collation.assume.ci}
> </argLine>
Actually that will not work. We can't pass parameters with dots in them because of an issue in Jenkins with Groovy scripting. Could you please make this
-Dcollation-ci=true
I've already hooked that into Jenkins, so when you commit it will start running the tests.
Ari
--
-------------------------->
Aristedes Maniatis
GPG fingerprint CBFB 84B4 738D 4E87 5E5C 5EFA EF6A 7D2E 3E49 102A
Re: svn commit: r1171003 - in /cayenne/main/trunk: ./ framework/cayenne-jdk1.5-unpublished/src/test/java/org/apache/cayenne/unit/
framework/cayenne-jdk1.5-unpublished/src/test/java/org/apache/cayenne/unit/di/server/
Posted by Aristedes Maniatis <ar...@maniatis.org>.
On 29/09/11 3:07 AM, Andrus Adamchik wrote:
> Ok, just figured out how to remove the "ci=true" option... Hopefully we won't be spammed with failure emails on every commit:-)
Sorry about that. I thought I already did it, but maybe I only dreamt that I did it...
Ari
--
-------------------------->
Aristedes Maniatis
GPG fingerprint CBFB 84B4 738D 4E87 5E5C 5EFA EF6A 7D2E 3E49 102A
Re: svn commit: r1171003 - in /cayenne/main/trunk: ./ framework/cayenne-jdk1.5-unpublished/src/test/java/org/apache/cayenne/unit/ framework/cayenne-jdk1.5-unpublished/src/test/java/org/apache/cayenne/unit/di/server/
Posted by Andrus Adamchik <an...@objectstyle.org>.
Ok, just figured out how to remove the "ci=true" option... Hopefully we won't be spammed with failure emails on every commit :-)
On Sep 16, 2011, at 3:04 AM, Aristedes Maniatis wrote:
> Yes... pretty like a set of traffic lights... But at least now we have a target to get all those dots blue. I need to research how we set each of those databases to ci, which I need to do for work anyway.
>
>
> Ari
>
>
> On 16/09/11 3:28 AM, Andrus Adamchik wrote:
>> http://people.apache.org/~aadamchik/tests.png
>>
>> I think this result demonstrates that you can't use collationCi=true when the underlying DB uses CS collations.
>>
>> Andrus
>>
>>
>> On Sep 15, 2011, at 5:14 PM, Aristedes Maniatis wrote:
>>
>>> OK, changed.
>>>
>>> On 15/09/11 7:17 PM, Andrus Adamchik wrote:
>>>> Let's be consistent here. We are using "camel case" property names for everything else to deal with Jenkins limitations ("cayenneTestConnection" etc). Let's keep using that instead of dashes... So the new property should probably be "collationCi".
>>>>
>>>> Andrus
>>>>
>>>> On Sep 15, 2011, at 11:55 AM, Aristedes Maniatis wrote:
>>>>
>>>>> On 15/09/11 6:27 PM, dkazimirchyk@apache.org wrote:
>>>>>> - -DcayenneTestConnection=${cayenneTestConnection} -Djava.net.preferIPv4Stack=true
>>>>>> + -DcayenneTestConnection=${cayenneTestConnection} -Djava.net.preferIPv4Stack=true -Dcayenne.runtime.db.collation.assume.ci=${cayenne.runtime.db.collation.assume.ci}
>>>>>> </argLine>
>>>>>
>>>>> Actually that will not work. We can't pass parameters with dots in them because of an issue in Jenkins with Groovy scripting. Could you please make this
>>>>>
>>>>> -Dcollation-ci=true
>>>>>
>>>>> I've already hooked that into Jenkins, so when you commit it will start running the tests.
>>>>>
>>>>>
>>>>> Ari
>>>>>
>>>>>
>>>>> --
>>>>> -------------------------->
>>>>> Aristedes Maniatis
>>>>> GPG fingerprint CBFB 84B4 738D 4E87 5E5C 5EFA EF6A 7D2E 3E49 102A
>>>>>
>>>>
>>>
>>> --
>>> -------------------------->
>>> Aristedes Maniatis
>>> GPG fingerprint CBFB 84B4 738D 4E87 5E5C 5EFA EF6A 7D2E 3E49 102A
>>>
>>
>
> --
> -------------------------->
> Aristedes Maniatis
> GPG fingerprint CBFB 84B4 738D 4E87 5E5C 5EFA EF6A 7D2E 3E49 102A
>
Re: svn commit: r1171003 - in /cayenne/main/trunk: ./ framework/cayenne-jdk1.5-unpublished/src/test/java/org/apache/cayenne/unit/ framework/cayenne-jdk1.5-unpublished/src/test/java/org/apache/cayenne/unit/di/server/
Posted by Andrus Adamchik <an...@objectstyle.org>.
Cool. If that requires more time, maybe we can deactivate the CI option on Jenkins for now, as it is pretty depressing seeing 6 failure emails on every commit :-)
Andrus
On Sep 16, 2011, at 3:04 AM, Aristedes Maniatis wrote:
> Yes... pretty like a set of traffic lights... But at least now we have a target to get all those dots blue. I need to research how we set each of those databases to ci, which I need to do for work anyway.
>
>
> Ari
>
>
> On 16/09/11 3:28 AM, Andrus Adamchik wrote:
>> http://people.apache.org/~aadamchik/tests.png
>>
>> I think this result demonstrates that you can't use collationCi=true when the underlying DB uses CS collations.
>>
>> Andrus
>>
>>
>> On Sep 15, 2011, at 5:14 PM, Aristedes Maniatis wrote:
>>
>>> OK, changed.
>>>
>>> On 15/09/11 7:17 PM, Andrus Adamchik wrote:
>>>> Let's be consistent here. We are using "camel case" property names for everything else to deal with Jenkins limitations ("cayenneTestConnection" etc). Let's keep using that instead of dashes... So the new property should probably be "collationCi".
>>>>
>>>> Andrus
>>>>
>>>> On Sep 15, 2011, at 11:55 AM, Aristedes Maniatis wrote:
>>>>
>>>>> On 15/09/11 6:27 PM, dkazimirchyk@apache.org wrote:
>>>>>> - -DcayenneTestConnection=${cayenneTestConnection} -Djava.net.preferIPv4Stack=true
>>>>>> + -DcayenneTestConnection=${cayenneTestConnection} -Djava.net.preferIPv4Stack=true -Dcayenne.runtime.db.collation.assume.ci=${cayenne.runtime.db.collation.assume.ci}
>>>>>> </argLine>
>>>>>
>>>>> Actually that will not work. We can't pass parameters with dots in them because of an issue in Jenkins with Groovy scripting. Could you please make this
>>>>>
>>>>> -Dcollation-ci=true
>>>>>
>>>>> I've already hooked that into Jenkins, so when you commit it will start running the tests.
>>>>>
>>>>>
>>>>> Ari
>>>>>
>>>>>
>>>>> --
>>>>> -------------------------->
>>>>> Aristedes Maniatis
>>>>> GPG fingerprint CBFB 84B4 738D 4E87 5E5C 5EFA EF6A 7D2E 3E49 102A
>>>>>
>>>>
>>>
>>> --
>>> -------------------------->
>>> Aristedes Maniatis
>>> GPG fingerprint CBFB 84B4 738D 4E87 5E5C 5EFA EF6A 7D2E 3E49 102A
>>>
>>
>
> --
> -------------------------->
> Aristedes Maniatis
> GPG fingerprint CBFB 84B4 738D 4E87 5E5C 5EFA EF6A 7D2E 3E49 102A
>
Re: svn commit: r1171003 - in /cayenne/main/trunk: ./ framework/cayenne-jdk1.5-unpublished/src/test/java/org/apache/cayenne/unit/
framework/cayenne-jdk1.5-unpublished/src/test/java/org/apache/cayenne/unit/di/server/
Posted by Aristedes Maniatis <ar...@maniatis.org>.
Yes... pretty like a set of traffic lights... But at least now we have a target to get all those dots blue. I need to research how we set each of those databases to ci, which I need to do for work anyway.
Ari
On 16/09/11 3:28 AM, Andrus Adamchik wrote:
> http://people.apache.org/~aadamchik/tests.png
>
> I think this result demonstrates that you can't use collationCi=true when the underlying DB uses CS collations.
>
> Andrus
>
>
> On Sep 15, 2011, at 5:14 PM, Aristedes Maniatis wrote:
>
>> OK, changed.
>>
>> On 15/09/11 7:17 PM, Andrus Adamchik wrote:
>>> Let's be consistent here. We are using "camel case" property names for everything else to deal with Jenkins limitations ("cayenneTestConnection" etc). Let's keep using that instead of dashes... So the new property should probably be "collationCi".
>>>
>>> Andrus
>>>
>>> On Sep 15, 2011, at 11:55 AM, Aristedes Maniatis wrote:
>>>
>>>> On 15/09/11 6:27 PM, dkazimirchyk@apache.org wrote:
>>>>> - -DcayenneTestConnection=${cayenneTestConnection} -Djava.net.preferIPv4Stack=true
>>>>> + -DcayenneTestConnection=${cayenneTestConnection} -Djava.net.preferIPv4Stack=true -Dcayenne.runtime.db.collation.assume.ci=${cayenne.runtime.db.collation.assume.ci}
>>>>> </argLine>
>>>>
>>>> Actually that will not work. We can't pass parameters with dots in them because of an issue in Jenkins with Groovy scripting. Could you please make this
>>>>
>>>> -Dcollation-ci=true
>>>>
>>>> I've already hooked that into Jenkins, so when you commit it will start running the tests.
>>>>
>>>>
>>>> Ari
>>>>
>>>>
>>>> --
>>>> -------------------------->
>>>> Aristedes Maniatis
>>>> GPG fingerprint CBFB 84B4 738D 4E87 5E5C 5EFA EF6A 7D2E 3E49 102A
>>>>
>>>
>>
>> --
>> -------------------------->
>> Aristedes Maniatis
>> GPG fingerprint CBFB 84B4 738D 4E87 5E5C 5EFA EF6A 7D2E 3E49 102A
>>
>
--
-------------------------->
Aristedes Maniatis
GPG fingerprint CBFB 84B4 738D 4E87 5E5C 5EFA EF6A 7D2E 3E49 102A
Re: svn commit: r1171003 - in /cayenne/main/trunk: ./ framework/cayenne-jdk1.5-unpublished/src/test/java/org/apache/cayenne/unit/ framework/cayenne-jdk1.5-unpublished/src/test/java/org/apache/cayenne/unit/di/server/
Posted by Andrus Adamchik <an...@objectstyle.org>.
http://people.apache.org/~aadamchik/tests.png
I think this result demonstrates that you can't use collationCi=true when the underlying DB uses CS collations.
Andrus
On Sep 15, 2011, at 5:14 PM, Aristedes Maniatis wrote:
> OK, changed.
>
> On 15/09/11 7:17 PM, Andrus Adamchik wrote:
>> Let's be consistent here. We are using "camel case" property names for everything else to deal with Jenkins limitations ("cayenneTestConnection" etc). Let's keep using that instead of dashes... So the new property should probably be "collationCi".
>>
>> Andrus
>>
>> On Sep 15, 2011, at 11:55 AM, Aristedes Maniatis wrote:
>>
>>> On 15/09/11 6:27 PM, dkazimirchyk@apache.org wrote:
>>>> - -DcayenneTestConnection=${cayenneTestConnection} -Djava.net.preferIPv4Stack=true
>>>> + -DcayenneTestConnection=${cayenneTestConnection} -Djava.net.preferIPv4Stack=true -Dcayenne.runtime.db.collation.assume.ci=${cayenne.runtime.db.collation.assume.ci}
>>>> </argLine>
>>>
>>> Actually that will not work. We can't pass parameters with dots in them because of an issue in Jenkins with Groovy scripting. Could you please make this
>>>
>>> -Dcollation-ci=true
>>>
>>> I've already hooked that into Jenkins, so when you commit it will start running the tests.
>>>
>>>
>>> Ari
>>>
>>>
>>> --
>>> -------------------------->
>>> Aristedes Maniatis
>>> GPG fingerprint CBFB 84B4 738D 4E87 5E5C 5EFA EF6A 7D2E 3E49 102A
>>>
>>
>
> --
> -------------------------->
> Aristedes Maniatis
> GPG fingerprint CBFB 84B4 738D 4E87 5E5C 5EFA EF6A 7D2E 3E49 102A
>
Re: svn commit: r1171003 - in /cayenne/main/trunk: ./ framework/cayenne-jdk1.5-unpublished/src/test/java/org/apache/cayenne/unit/
framework/cayenne-jdk1.5-unpublished/src/test/java/org/apache/cayenne/unit/di/server/
Posted by Aristedes Maniatis <ar...@maniatis.org>.
OK, changed.
On 15/09/11 7:17 PM, Andrus Adamchik wrote:
> Let's be consistent here. We are using "camel case" property names for everything else to deal with Jenkins limitations ("cayenneTestConnection" etc). Let's keep using that instead of dashes... So the new property should probably be "collationCi".
>
> Andrus
>
> On Sep 15, 2011, at 11:55 AM, Aristedes Maniatis wrote:
>
>> On 15/09/11 6:27 PM, dkazimirchyk@apache.org wrote:
>>> - -DcayenneTestConnection=${cayenneTestConnection} -Djava.net.preferIPv4Stack=true
>>> + -DcayenneTestConnection=${cayenneTestConnection} -Djava.net.preferIPv4Stack=true -Dcayenne.runtime.db.collation.assume.ci=${cayenne.runtime.db.collation.assume.ci}
>>> </argLine>
>>
>> Actually that will not work. We can't pass parameters with dots in them because of an issue in Jenkins with Groovy scripting. Could you please make this
>>
>> -Dcollation-ci=true
>>
>> I've already hooked that into Jenkins, so when you commit it will start running the tests.
>>
>>
>> Ari
>>
>>
>> --
>> -------------------------->
>> Aristedes Maniatis
>> GPG fingerprint CBFB 84B4 738D 4E87 5E5C 5EFA EF6A 7D2E 3E49 102A
>>
>
--
-------------------------->
Aristedes Maniatis
GPG fingerprint CBFB 84B4 738D 4E87 5E5C 5EFA EF6A 7D2E 3E49 102A
Re: svn commit: r1171003 - in /cayenne/main/trunk: ./ framework/cayenne-jdk1.5-unpublished/src/test/java/org/apache/cayenne/unit/ framework/cayenne-jdk1.5-unpublished/src/test/java/org/apache/cayenne/unit/di/server/
Posted by Andrus Adamchik <an...@objectstyle.org>.
Let's be consistent here. We are using "camel case" property names for everything else to deal with Jenkins limitations ("cayenneTestConnection" etc). Let's keep using that instead of dashes... So the new property should probably be "collationCi".
Andrus
On Sep 15, 2011, at 11:55 AM, Aristedes Maniatis wrote:
> On 15/09/11 6:27 PM, dkazimirchyk@apache.org wrote:
>> - -DcayenneTestConnection=${cayenneTestConnection} -Djava.net.preferIPv4Stack=true
>> + -DcayenneTestConnection=${cayenneTestConnection} -Djava.net.preferIPv4Stack=true -Dcayenne.runtime.db.collation.assume.ci=${cayenne.runtime.db.collation.assume.ci}
>> </argLine>
>
> Actually that will not work. We can't pass parameters with dots in them because of an issue in Jenkins with Groovy scripting. Could you please make this
>
> -Dcollation-ci=true
>
> I've already hooked that into Jenkins, so when you commit it will start running the tests.
>
>
> Ari
>
>
> --
> -------------------------->
> Aristedes Maniatis
> GPG fingerprint CBFB 84B4 738D 4E87 5E5C 5EFA EF6A 7D2E 3E49 102A
>