You are viewing a plain text version of this content. The canonical link for it is here.
Posted to users@tomee.apache.org by hwaastad <he...@waastad.org> on 2014/06/05 11:31:42 UTC

TomEE 1.7.0 and quartz properties

Has there been any change in quartz properties in tomee 1.7.0

I get a org.apache.openejb.quartz.SchedulerConfigException: Thread count
must be > 0

I'm setting property:
props.setProperty("org.quartz.threadPool.threadCount", "3");

br hw




--
View this message in context: http://openejb.979440.n4.nabble.com/TomEE-1-7-0-and-quartz-properties-tp4669891.html
Sent from the OpenEJB User mailing list archive at Nabble.com.

Re: TomEE 1.7.0 and quartz properties

Posted by Romain Manni-Bucau <rm...@gmail.com>.
cause default key is openejb.timer.pool.size when using a shared scheduler no?


Romain Manni-Bucau
Twitter: @rmannibucau
Blog: http://rmannibucau.wordpress.com/
LinkedIn: http://fr.linkedin.com/in/rmannibucau
Github: https://github.com/rmannibucau


2014-06-05 17:44 GMT+02:00 Andy Gumbrecht <ag...@tomitribe.com>:
> Sure, I remember it, just don't get why the original property does not work?
> HW shouldn't have needed to change anything. I'm sure we need a test.
>
> Andy.
>
>
> On 05/06/2014 17:38, Romain Manni-Bucau wrote:
>>
>> @Andy: we adjust it in EjbTimerServiceImpl#putAll, we just don't log
>> we adjusted them
>>
>>
>> Romain Manni-Bucau
>> Twitter: @rmannibucau
>> Blog: http://rmannibucau.wordpress.com/
>> LinkedIn: http://fr.linkedin.com/in/rmannibucau
>> Github: https://github.com/rmannibucau
>>
>>
>> 2014-06-05 17:35 GMT+02:00 Andy Gumbrecht <ag...@tomitribe.com>:
>>>
>>> Hi H,
>>>
>>> That sounds like a regression to me - You shouldn't need to change that
>>> property?
>>>
>>> I've flagged it for a review here:
>>> https://issues.apache.org/jira/browse/TOMEE-1243
>>>
>>> Andy.
>>>
>>>
>>> On 05/06/2014 11:53, hwaastad wrote:
>>>>
>>>> Yepp.
>>>>
>>>> I've realigned all my quartz properties to the openenjb shaded version
>>>> and
>>>> it seems to be OK.
>>>>
>>>> No errors....yet :-)
>>>>
>>>> br hw
>>>>
>>>>
>>>>
>>>> --
>>>> View this message in context:
>>>>
>>>> http://openejb.979440.n4.nabble.com/TomEE-1-7-0-and-quartz-properties-tp4669891p4669894.html
>>>> Sent from the OpenEJB User mailing list archive at Nabble.com.
>>>
>>>
>>> --
>>>    Andy Gumbrecht
>>>
>>>    http://www.tomitribe.com
>>>    agumbrecht@tomitribe.com
>>>    https://twitter.com/AndyGeeDe
>>>
>>>    TomEE treibt Tomitribe! | http://tomee.apache.org
>>>
>
> --
>   Andy Gumbrecht
>
>   http://www.tomitribe.com
>   agumbrecht@tomitribe.com
>   https://twitter.com/AndyGeeDe
>
>   TomEE treibt Tomitribe! | http://tomee.apache.org
>

Re: TomEE 1.7.0 and quartz properties

Posted by Andy Gumbrecht <ag...@tomitribe.com>.
Sure, I remember it, just don't get why the original property does not 
work? HW shouldn't have needed to change anything. I'm sure we need a test.

Andy.

On 05/06/2014 17:38, Romain Manni-Bucau wrote:
> @Andy: we adjust it in EjbTimerServiceImpl#putAll, we just don't log
> we adjusted them
>
>
> Romain Manni-Bucau
> Twitter: @rmannibucau
> Blog: http://rmannibucau.wordpress.com/
> LinkedIn: http://fr.linkedin.com/in/rmannibucau
> Github: https://github.com/rmannibucau
>
>
> 2014-06-05 17:35 GMT+02:00 Andy Gumbrecht <ag...@tomitribe.com>:
>> Hi H,
>>
>> That sounds like a regression to me - You shouldn't need to change that
>> property?
>>
>> I've flagged it for a review here:
>> https://issues.apache.org/jira/browse/TOMEE-1243
>>
>> Andy.
>>
>>
>> On 05/06/2014 11:53, hwaastad wrote:
>>> Yepp.
>>>
>>> I've realigned all my quartz properties to the openenjb shaded version and
>>> it seems to be OK.
>>>
>>> No errors....yet :-)
>>>
>>> br hw
>>>
>>>
>>>
>>> --
>>> View this message in context:
>>> http://openejb.979440.n4.nabble.com/TomEE-1-7-0-and-quartz-properties-tp4669891p4669894.html
>>> Sent from the OpenEJB User mailing list archive at Nabble.com.
>>
>> --
>>    Andy Gumbrecht
>>
>>    http://www.tomitribe.com
>>    agumbrecht@tomitribe.com
>>    https://twitter.com/AndyGeeDe
>>
>>    TomEE treibt Tomitribe! | http://tomee.apache.org
>>

-- 
   Andy Gumbrecht

   http://www.tomitribe.com
   agumbrecht@tomitribe.com
   https://twitter.com/AndyGeeDe

   TomEE treibt Tomitribe! | http://tomee.apache.org


Re: TomEE 1.7.0 and quartz properties

Posted by Romain Manni-Bucau <rm...@gmail.com>.
@Andy: we adjust it in EjbTimerServiceImpl#putAll, we just don't log
we adjusted them


Romain Manni-Bucau
Twitter: @rmannibucau
Blog: http://rmannibucau.wordpress.com/
LinkedIn: http://fr.linkedin.com/in/rmannibucau
Github: https://github.com/rmannibucau


2014-06-05 17:35 GMT+02:00 Andy Gumbrecht <ag...@tomitribe.com>:
> Hi H,
>
> That sounds like a regression to me - You shouldn't need to change that
> property?
>
> I've flagged it for a review here:
> https://issues.apache.org/jira/browse/TOMEE-1243
>
> Andy.
>
>
> On 05/06/2014 11:53, hwaastad wrote:
>>
>> Yepp.
>>
>> I've realigned all my quartz properties to the openenjb shaded version and
>> it seems to be OK.
>>
>> No errors....yet :-)
>>
>> br hw
>>
>>
>>
>> --
>> View this message in context:
>> http://openejb.979440.n4.nabble.com/TomEE-1-7-0-and-quartz-properties-tp4669891p4669894.html
>> Sent from the OpenEJB User mailing list archive at Nabble.com.
>
>
> --
>   Andy Gumbrecht
>
>   http://www.tomitribe.com
>   agumbrecht@tomitribe.com
>   https://twitter.com/AndyGeeDe
>
>   TomEE treibt Tomitribe! | http://tomee.apache.org
>

Re: TomEE 1.7.0 and quartz properties

Posted by Andy Gumbrecht <ag...@tomitribe.com>.
Hi H,

That sounds like a regression to me - You shouldn't need to change that 
property?

I've flagged it for a review here: 
https://issues.apache.org/jira/browse/TOMEE-1243

Andy.

On 05/06/2014 11:53, hwaastad wrote:
> Yepp.
>
> I've realigned all my quartz properties to the openenjb shaded version and
> it seems to be OK.
>
> No errors....yet :-)
>
> br hw
>
>
>
> --
> View this message in context: http://openejb.979440.n4.nabble.com/TomEE-1-7-0-and-quartz-properties-tp4669891p4669894.html
> Sent from the OpenEJB User mailing list archive at Nabble.com.

-- 
   Andy Gumbrecht

   http://www.tomitribe.com
   agumbrecht@tomitribe.com
   https://twitter.com/AndyGeeDe

   TomEE treibt Tomitribe! | http://tomee.apache.org


Re: TomEE 1.7.0 and quartz properties

Posted by hwaastad <he...@waastad.org>.
Yepp.

I've realigned all my quartz properties to the openenjb shaded version and
it seems to be OK.

No errors....yet :-)

br hw



--
View this message in context: http://openejb.979440.n4.nabble.com/TomEE-1-7-0-and-quartz-properties-tp4669891p4669894.html
Sent from the OpenEJB User mailing list archive at Nabble.com.

Re: TomEE 1.7.0 and quartz properties

Posted by Romain Manni-Bucau <rm...@gmail.com>.
PS: compatibility should have been preserved


Romain Manni-Bucau
Twitter: @rmannibucau
Blog: http://rmannibucau.wordpress.com/
LinkedIn: http://fr.linkedin.com/in/rmannibucau
Github: https://github.com/rmannibucau


2014-06-05 11:39 GMT+02:00 Romain Manni-Bucau <rm...@gmail.com>:
> Hi
>
> yes, since we shaded quartz org.quartz is now org.apache.openejb.quartz.
>
> We'll restore the compatibility pretty soon but with a warning asking
> the user to migrate
>
>
> Romain Manni-Bucau
> Twitter: @rmannibucau
> Blog: http://rmannibucau.wordpress.com/
> LinkedIn: http://fr.linkedin.com/in/rmannibucau
> Github: https://github.com/rmannibucau
>
>
> 2014-06-05 11:31 GMT+02:00 hwaastad <he...@waastad.org>:
>> Has there been any change in quartz properties in tomee 1.7.0
>>
>> I get a org.apache.openejb.quartz.SchedulerConfigException: Thread count
>> must be > 0
>>
>> I'm setting property:
>> props.setProperty("org.quartz.threadPool.threadCount", "3");
>>
>> br hw
>>
>>
>>
>>
>> --
>> View this message in context: http://openejb.979440.n4.nabble.com/TomEE-1-7-0-and-quartz-properties-tp4669891.html
>> Sent from the OpenEJB User mailing list archive at Nabble.com.

Re: TomEE 1.7.0 and quartz properties

Posted by Romain Manni-Bucau <rm...@gmail.com>.
Hi

yes, since we shaded quartz org.quartz is now org.apache.openejb.quartz.

We'll restore the compatibility pretty soon but with a warning asking
the user to migrate


Romain Manni-Bucau
Twitter: @rmannibucau
Blog: http://rmannibucau.wordpress.com/
LinkedIn: http://fr.linkedin.com/in/rmannibucau
Github: https://github.com/rmannibucau


2014-06-05 11:31 GMT+02:00 hwaastad <he...@waastad.org>:
> Has there been any change in quartz properties in tomee 1.7.0
>
> I get a org.apache.openejb.quartz.SchedulerConfigException: Thread count
> must be > 0
>
> I'm setting property:
> props.setProperty("org.quartz.threadPool.threadCount", "3");
>
> br hw
>
>
>
>
> --
> View this message in context: http://openejb.979440.n4.nabble.com/TomEE-1-7-0-and-quartz-properties-tp4669891.html
> Sent from the OpenEJB User mailing list archive at Nabble.com.