You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by David E Jones <jo...@undersunconsulting.com> on 2008/04/09 04:29:53 UTC
Re: svn commit: r646132 - /ofbiz/trunk/applications/order/servicedef/services_cart.xml
Actually, transactions ARE important for reading as well as writing,
and should pretty much always be used.
What was the problem you had that this change is supposed to solve?
-David
On Apr 8, 2008, at 5:21 PM, sichen@apache.org wrote:
> Author: sichen
> Date: Tue Apr 8 16:21:14 2008
> New Revision: 646132
>
> URL: http://svn.apache.org/viewvc?rev=646132&view=rev
> Log:
> turning off transactions for loading of cart from quote, since there
> does not seem to be any data being stored, and having a transaction
> consumes resources unnecessarily and could cause it to fail if there
> are many items on a quote
>
> Modified:
> ofbiz/trunk/applications/order/servicedef/services_cart.xml
>
> Modified: ofbiz/trunk/applications/order/servicedef/services_cart.xml
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services_cart.xml?rev=646132&r1=646131&r2=646132&view=diff
> =
> =
> =
> =
> =
> =
> =
> =
> ======================================================================
> --- ofbiz/trunk/applications/order/servicedef/services_cart.xml
> (original)
> +++ ofbiz/trunk/applications/order/servicedef/services_cart.xml Tue
> Apr 8 16:21:14 2008
> @@ -103,7 +103,7 @@
> </service>
>
> <service name="loadCartFromQuote" engine="java" auth="true"
> -
> location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
> invoke="loadCartFromQuote">
> +
> location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
> invoke="loadCartFromQuote" use-transaction="false">
> <description>Create a ShoppingCart Object based on an
> existing quote. If applyQuoteAdjustments is set to false then
> standard cart adjustments are generated.</description>
> <attribute name="quoteId" type="String" mode="IN"
> optional="false"/>
> <attribute name="applyQuoteAdjustments" type="String"
> mode="IN" optional="true"/>
>
>
Re: svn commit: r646132 - /ofbiz/trunk/applications/order/servicedef/services_cart.xml
Posted by Jacques Le Roux <ja...@les7arts.com>.
Yes, 4 days after this seems pretty reasonnable...
Jacques
From: "Scott Gray" <le...@gmail.com>
> +1
>
> Scott
>
> On 13/04/2008, Jacopo Cappellato <ja...@gmail.com> wrote:
>>
>> No feedback on this?
>> However, a committer should always be around to explain and provide
>> further details if there are concerns or questions on his commits.
>>
>> Should we revert this commit?
>>
>> Jacopo
>>
>> On Apr 9, 2008, at 4:29 AM, David E Jones wrote:
>>
>> >
>> > Actually, transactions ARE important for reading as well as writing, and
>> > should pretty much always be used.
>> >
>> > What was the problem you had that this change is supposed to solve?
>> >
>> > -David
>> >
>> >
>> > On Apr 8, 2008, at 5:21 PM, sichen@apache.org wrote:
>> >
>> > > Author: sichen
>> > > Date: Tue Apr 8 16:21:14 2008
>> > > New Revision: 646132
>> > >
>> > > URL: http://svn.apache.org/viewvc?rev=646132&view=rev
>> > > Log:
>> > > turning off transactions for loading of cart from quote, since there
>> > > does not seem to be any data being stored, and having a transaction consumes
>> > > resources unnecessarily and could cause it to fail if there are many items
>> > > on a quote
>> > >
>> > > Modified:
>> > > ofbiz/trunk/applications/order/servicedef/services_cart.xml
>> > >
>> > > Modified: ofbiz/trunk/applications/order/servicedef/services_cart.xml
>> > > URL:
>> > > http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services_cart.xml?rev=646132&r1=646131&r2=646132&view=diff
>> > >
>> > > ==============================================================================
>> > > --- ofbiz/trunk/applications/order/servicedef/services_cart.xml
>> > > (original)
>> > > +++ ofbiz/trunk/applications/order/servicedef/services_cart.xml Tue
>> > > Apr 8 16:21:14 2008
>> > > @@ -103,7 +103,7 @@
>> > > </service>
>> > >
>> > > <service name="loadCartFromQuote" engine="java" auth="true"
>> > > -
>> > > location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
>> > > invoke="loadCartFromQuote">
>> > > +
>> > > location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
>> > > invoke="loadCartFromQuote" use-transaction="false">
>> > > <description>Create a ShoppingCart Object based on an existing
>> > > quote. If applyQuoteAdjustments is set to false then standard cart
>> > > adjustments are generated.</description>
>> > > <attribute name="quoteId" type="String" mode="IN"
>> > > optional="false"/>
>> > > <attribute name="applyQuoteAdjustments" type="String" mode="IN"
>> > > optional="true"/>
>> > >
>> > >
>> > >
>> >
>>
>
Re: svn commit: r646132 - /ofbiz/trunk/applications/order/servicedef/services_cart.xml
Posted by Scott Gray <le...@gmail.com>.
+1
Scott
On 13/04/2008, Jacopo Cappellato <ja...@gmail.com> wrote:
>
> No feedback on this?
> However, a committer should always be around to explain and provide
> further details if there are concerns or questions on his commits.
>
> Should we revert this commit?
>
> Jacopo
>
> On Apr 9, 2008, at 4:29 AM, David E Jones wrote:
>
> >
> > Actually, transactions ARE important for reading as well as writing, and
> > should pretty much always be used.
> >
> > What was the problem you had that this change is supposed to solve?
> >
> > -David
> >
> >
> > On Apr 8, 2008, at 5:21 PM, sichen@apache.org wrote:
> >
> > > Author: sichen
> > > Date: Tue Apr 8 16:21:14 2008
> > > New Revision: 646132
> > >
> > > URL: http://svn.apache.org/viewvc?rev=646132&view=rev
> > > Log:
> > > turning off transactions for loading of cart from quote, since there
> > > does not seem to be any data being stored, and having a transaction consumes
> > > resources unnecessarily and could cause it to fail if there are many items
> > > on a quote
> > >
> > > Modified:
> > > ofbiz/trunk/applications/order/servicedef/services_cart.xml
> > >
> > > Modified: ofbiz/trunk/applications/order/servicedef/services_cart.xml
> > > URL:
> > > http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services_cart.xml?rev=646132&r1=646131&r2=646132&view=diff
> > >
> > > ==============================================================================
> > > --- ofbiz/trunk/applications/order/servicedef/services_cart.xml
> > > (original)
> > > +++ ofbiz/trunk/applications/order/servicedef/services_cart.xml Tue
> > > Apr 8 16:21:14 2008
> > > @@ -103,7 +103,7 @@
> > > </service>
> > >
> > > <service name="loadCartFromQuote" engine="java" auth="true"
> > > -
> > > location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
> > > invoke="loadCartFromQuote">
> > > +
> > > location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
> > > invoke="loadCartFromQuote" use-transaction="false">
> > > <description>Create a ShoppingCart Object based on an existing
> > > quote. If applyQuoteAdjustments is set to false then standard cart
> > > adjustments are generated.</description>
> > > <attribute name="quoteId" type="String" mode="IN"
> > > optional="false"/>
> > > <attribute name="applyQuoteAdjustments" type="String" mode="IN"
> > > optional="true"/>
> > >
> > >
> > >
> >
>
Re: svn commit: r646132 - /ofbiz/trunk/applications/order/servicedef/services_cart.xml
Posted by Jacopo Cappellato <ja...@gmail.com>.
This is finally reverted in rev. 651193
Jacopo
On Apr 14, 2008, at 7:05 AM, David E Jones wrote:
>
> +1 from me on the revert as well (making it clear as it is only
> implied in my initial reply).
>
> -David
>
>
> On Apr 12, 2008, at 11:04 PM, Jacopo Cappellato wrote:
>> No feedback on this?
>> However, a committer should always be around to explain and provide
>> further details if there are concerns or questions on his commits.
>>
>> Should we revert this commit?
>>
>> Jacopo
>>
>> On Apr 9, 2008, at 4:29 AM, David E Jones wrote:
>>>
>>> Actually, transactions ARE important for reading as well as
>>> writing, and should pretty much always be used.
>>>
>>> What was the problem you had that this change is supposed to solve?
>>>
>>> -David
>>>
>>>
>>> On Apr 8, 2008, at 5:21 PM, sichen@apache.org wrote:
>>>> Author: sichen
>>>> Date: Tue Apr 8 16:21:14 2008
>>>> New Revision: 646132
>>>>
>>>> URL: http://svn.apache.org/viewvc?rev=646132&view=rev
>>>> Log:
>>>> turning off transactions for loading of cart from quote, since
>>>> there does not seem to be any data being stored, and having a
>>>> transaction consumes resources unnecessarily and could cause it
>>>> to fail if there are many items on a quote
>>>>
>>>> Modified:
>>>> ofbiz/trunk/applications/order/servicedef/services_cart.xml
>>>>
>>>> Modified: ofbiz/trunk/applications/order/servicedef/
>>>> services_cart.xml
>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services_cart.xml?rev=646132&r1=646131&r2=646132&view=diff
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> =
>>>> ===================================================================
>>>> --- ofbiz/trunk/applications/order/servicedef/services_cart.xml
>>>> (original)
>>>> +++ ofbiz/trunk/applications/order/servicedef/services_cart.xml
>>>> Tue Apr 8 16:21:14 2008
>>>> @@ -103,7 +103,7 @@
>>>> </service>
>>>>
>>>> <service name="loadCartFromQuote" engine="java" auth="true"
>>>> -
>>>> location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
>>>> invoke="loadCartFromQuote">
>>>> +
>>>> location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
>>>> invoke="loadCartFromQuote" use-transaction="false">
>>>> <description>Create a ShoppingCart Object based on an
>>>> existing quote. If applyQuoteAdjustments is set to false then
>>>> standard cart adjustments are generated.</description>
>>>> <attribute name="quoteId" type="String" mode="IN"
>>>> optional="false"/>
>>>> <attribute name="applyQuoteAdjustments" type="String"
>>>> mode="IN" optional="true"/>
>>>>
>>>>
>>>
>>
>
Re: svn commit: r646132 - /ofbiz/trunk/applications/order/servicedef/services_cart.xml
Posted by David E Jones <jo...@hotwaxmedia.com>.
+1 from me on the revert as well (making it clear as it is only
implied in my initial reply).
-David
On Apr 12, 2008, at 11:04 PM, Jacopo Cappellato wrote:
> No feedback on this?
> However, a committer should always be around to explain and provide
> further details if there are concerns or questions on his commits.
>
> Should we revert this commit?
>
> Jacopo
>
> On Apr 9, 2008, at 4:29 AM, David E Jones wrote:
>>
>> Actually, transactions ARE important for reading as well as
>> writing, and should pretty much always be used.
>>
>> What was the problem you had that this change is supposed to solve?
>>
>> -David
>>
>>
>> On Apr 8, 2008, at 5:21 PM, sichen@apache.org wrote:
>>> Author: sichen
>>> Date: Tue Apr 8 16:21:14 2008
>>> New Revision: 646132
>>>
>>> URL: http://svn.apache.org/viewvc?rev=646132&view=rev
>>> Log:
>>> turning off transactions for loading of cart from quote, since
>>> there does not seem to be any data being stored, and having a
>>> transaction consumes resources unnecessarily and could cause it to
>>> fail if there are many items on a quote
>>>
>>> Modified:
>>> ofbiz/trunk/applications/order/servicedef/services_cart.xml
>>>
>>> Modified: ofbiz/trunk/applications/order/servicedef/
>>> services_cart.xml
>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services_cart.xml?rev=646132&r1=646131&r2=646132&view=diff
>>> =
>>> =
>>> =
>>> =
>>> =
>>> =
>>> =
>>> =
>>> =
>>> =
>>> ====================================================================
>>> --- ofbiz/trunk/applications/order/servicedef/services_cart.xml
>>> (original)
>>> +++ ofbiz/trunk/applications/order/servicedef/services_cart.xml
>>> Tue Apr 8 16:21:14 2008
>>> @@ -103,7 +103,7 @@
>>> </service>
>>>
>>> <service name="loadCartFromQuote" engine="java" auth="true"
>>> -
>>> location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
>>> invoke="loadCartFromQuote">
>>> +
>>> location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
>>> invoke="loadCartFromQuote" use-transaction="false">
>>> <description>Create a ShoppingCart Object based on an
>>> existing quote. If applyQuoteAdjustments is set to false then
>>> standard cart adjustments are generated.</description>
>>> <attribute name="quoteId" type="String" mode="IN"
>>> optional="false"/>
>>> <attribute name="applyQuoteAdjustments" type="String"
>>> mode="IN" optional="true"/>
>>>
>>>
>>
>
Re: svn commit: r646132 - /ofbiz/trunk/applications/order/servicedef/services_cart.xml
Posted by David E Jones <jo...@hotwaxmedia.com>.
+1 from me on the revert as well (making it clear as it is only
implied in my initial reply).
-David
On Apr 12, 2008, at 11:04 PM, Jacopo Cappellato wrote:
> No feedback on this?
> However, a committer should always be around to explain and provide
> further details if there are concerns or questions on his commits.
>
> Should we revert this commit?
>
> Jacopo
>
> On Apr 9, 2008, at 4:29 AM, David E Jones wrote:
>>
>> Actually, transactions ARE important for reading as well as
>> writing, and should pretty much always be used.
>>
>> What was the problem you had that this change is supposed to solve?
>>
>> -David
>>
>>
>> On Apr 8, 2008, at 5:21 PM, sichen@apache.org wrote:
>>> Author: sichen
>>> Date: Tue Apr 8 16:21:14 2008
>>> New Revision: 646132
>>>
>>> URL: http://svn.apache.org/viewvc?rev=646132&view=rev
>>> Log:
>>> turning off transactions for loading of cart from quote, since
>>> there does not seem to be any data being stored, and having a
>>> transaction consumes resources unnecessarily and could cause it to
>>> fail if there are many items on a quote
>>>
>>> Modified:
>>> ofbiz/trunk/applications/order/servicedef/services_cart.xml
>>>
>>> Modified: ofbiz/trunk/applications/order/servicedef/
>>> services_cart.xml
>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services_cart.xml?rev=646132&r1=646131&r2=646132&view=diff
>>> =
>>> =
>>> =
>>> =
>>> =
>>> =
>>> =
>>> =
>>> =
>>> =
>>> ====================================================================
>>> --- ofbiz/trunk/applications/order/servicedef/services_cart.xml
>>> (original)
>>> +++ ofbiz/trunk/applications/order/servicedef/services_cart.xml
>>> Tue Apr 8 16:21:14 2008
>>> @@ -103,7 +103,7 @@
>>> </service>
>>>
>>> <service name="loadCartFromQuote" engine="java" auth="true"
>>> -
>>> location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
>>> invoke="loadCartFromQuote">
>>> +
>>> location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
>>> invoke="loadCartFromQuote" use-transaction="false">
>>> <description>Create a ShoppingCart Object based on an
>>> existing quote. If applyQuoteAdjustments is set to false then
>>> standard cart adjustments are generated.</description>
>>> <attribute name="quoteId" type="String" mode="IN"
>>> optional="false"/>
>>> <attribute name="applyQuoteAdjustments" type="String"
>>> mode="IN" optional="true"/>
>>>
>>>
>>
>
Re: svn commit: r646132 - /ofbiz/trunk/applications/order/servicedef/services_cart.xml
Posted by Scott Gray <le...@gmail.com>.
+1
Scott
On 13/04/2008, Jacopo Cappellato <ja...@gmail.com> wrote:
>
> No feedback on this?
> However, a committer should always be around to explain and provide
> further details if there are concerns or questions on his commits.
>
> Should we revert this commit?
>
> Jacopo
>
> On Apr 9, 2008, at 4:29 AM, David E Jones wrote:
>
> >
> > Actually, transactions ARE important for reading as well as writing, and
> > should pretty much always be used.
> >
> > What was the problem you had that this change is supposed to solve?
> >
> > -David
> >
> >
> > On Apr 8, 2008, at 5:21 PM, sichen@apache.org wrote:
> >
> > > Author: sichen
> > > Date: Tue Apr 8 16:21:14 2008
> > > New Revision: 646132
> > >
> > > URL: http://svn.apache.org/viewvc?rev=646132&view=rev
> > > Log:
> > > turning off transactions for loading of cart from quote, since there
> > > does not seem to be any data being stored, and having a transaction consumes
> > > resources unnecessarily and could cause it to fail if there are many items
> > > on a quote
> > >
> > > Modified:
> > > ofbiz/trunk/applications/order/servicedef/services_cart.xml
> > >
> > > Modified: ofbiz/trunk/applications/order/servicedef/services_cart.xml
> > > URL:
> > > http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services_cart.xml?rev=646132&r1=646131&r2=646132&view=diff
> > >
> > > ==============================================================================
> > > --- ofbiz/trunk/applications/order/servicedef/services_cart.xml
> > > (original)
> > > +++ ofbiz/trunk/applications/order/servicedef/services_cart.xml Tue
> > > Apr 8 16:21:14 2008
> > > @@ -103,7 +103,7 @@
> > > </service>
> > >
> > > <service name="loadCartFromQuote" engine="java" auth="true"
> > > -
> > > location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
> > > invoke="loadCartFromQuote">
> > > +
> > > location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
> > > invoke="loadCartFromQuote" use-transaction="false">
> > > <description>Create a ShoppingCart Object based on an existing
> > > quote. If applyQuoteAdjustments is set to false then standard cart
> > > adjustments are generated.</description>
> > > <attribute name="quoteId" type="String" mode="IN"
> > > optional="false"/>
> > > <attribute name="applyQuoteAdjustments" type="String" mode="IN"
> > > optional="true"/>
> > >
> > >
> > >
> >
>
Re: svn commit: r646132 - /ofbiz/trunk/applications/order/servicedef/services_cart.xml
Posted by Jacopo Cappellato <ja...@gmail.com>.
No feedback on this?
However, a committer should always be around to explain and provide
further details if there are concerns or questions on his commits.
Should we revert this commit?
Jacopo
On Apr 9, 2008, at 4:29 AM, David E Jones wrote:
>
> Actually, transactions ARE important for reading as well as writing,
> and should pretty much always be used.
>
> What was the problem you had that this change is supposed to solve?
>
> -David
>
>
> On Apr 8, 2008, at 5:21 PM, sichen@apache.org wrote:
>> Author: sichen
>> Date: Tue Apr 8 16:21:14 2008
>> New Revision: 646132
>>
>> URL: http://svn.apache.org/viewvc?rev=646132&view=rev
>> Log:
>> turning off transactions for loading of cart from quote, since
>> there does not seem to be any data being stored, and having a
>> transaction consumes resources unnecessarily and could cause it to
>> fail if there are many items on a quote
>>
>> Modified:
>> ofbiz/trunk/applications/order/servicedef/services_cart.xml
>>
>> Modified: ofbiz/trunk/applications/order/servicedef/services_cart.xml
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services_cart.xml?rev=646132&r1=646131&r2=646132&view=diff
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> =====================================================================
>> --- ofbiz/trunk/applications/order/servicedef/services_cart.xml
>> (original)
>> +++ ofbiz/trunk/applications/order/servicedef/services_cart.xml Tue
>> Apr 8 16:21:14 2008
>> @@ -103,7 +103,7 @@
>> </service>
>>
>> <service name="loadCartFromQuote" engine="java" auth="true"
>> -
>> location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
>> invoke="loadCartFromQuote">
>> +
>> location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
>> invoke="loadCartFromQuote" use-transaction="false">
>> <description>Create a ShoppingCart Object based on an
>> existing quote. If applyQuoteAdjustments is set to false then
>> standard cart adjustments are generated.</description>
>> <attribute name="quoteId" type="String" mode="IN"
>> optional="false"/>
>> <attribute name="applyQuoteAdjustments" type="String"
>> mode="IN" optional="true"/>
>>
>>
>
Re: svn commit: r646132 - /ofbiz/trunk/applications/order/servicedef/services_cart.xml
Posted by Jacopo Cappellato <ja...@gmail.com>.
No feedback on this?
However, a committer should always be around to explain and provide
further details if there are concerns or questions on his commits.
Should we revert this commit?
Jacopo
On Apr 9, 2008, at 4:29 AM, David E Jones wrote:
>
> Actually, transactions ARE important for reading as well as writing,
> and should pretty much always be used.
>
> What was the problem you had that this change is supposed to solve?
>
> -David
>
>
> On Apr 8, 2008, at 5:21 PM, sichen@apache.org wrote:
>> Author: sichen
>> Date: Tue Apr 8 16:21:14 2008
>> New Revision: 646132
>>
>> URL: http://svn.apache.org/viewvc?rev=646132&view=rev
>> Log:
>> turning off transactions for loading of cart from quote, since
>> there does not seem to be any data being stored, and having a
>> transaction consumes resources unnecessarily and could cause it to
>> fail if there are many items on a quote
>>
>> Modified:
>> ofbiz/trunk/applications/order/servicedef/services_cart.xml
>>
>> Modified: ofbiz/trunk/applications/order/servicedef/services_cart.xml
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services_cart.xml?rev=646132&r1=646131&r2=646132&view=diff
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> =====================================================================
>> --- ofbiz/trunk/applications/order/servicedef/services_cart.xml
>> (original)
>> +++ ofbiz/trunk/applications/order/servicedef/services_cart.xml Tue
>> Apr 8 16:21:14 2008
>> @@ -103,7 +103,7 @@
>> </service>
>>
>> <service name="loadCartFromQuote" engine="java" auth="true"
>> -
>> location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
>> invoke="loadCartFromQuote">
>> +
>> location="org.ofbiz.order.shoppingcart.ShoppingCartServices"
>> invoke="loadCartFromQuote" use-transaction="false">
>> <description>Create a ShoppingCart Object based on an
>> existing quote. If applyQuoteAdjustments is set to false then
>> standard cart adjustments are generated.</description>
>> <attribute name="quoteId" type="String" mode="IN"
>> optional="false"/>
>> <attribute name="applyQuoteAdjustments" type="String"
>> mode="IN" optional="true"/>
>>
>>
>