Re: svn commit: r646132 - /ofbiz/trunk/applications/order/servicedef/services_cart.xml
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, [EMAIL PROTECTED] wrote: Author: sichen Date: Tue Apr 8 16:21:14 2008 New Revision: 646132 URL: http://svn.apache.org/viewvc?rev=646132view=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=646132r1=646131r2=646132view=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 descriptionCreate 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
Yes, 4 days after this seems pretty reasonnable... Jacques From: Scott Gray [EMAIL PROTECTED] +1 Scott On 13/04/2008, Jacopo Cappellato [EMAIL PROTECTED] 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, [EMAIL PROTECTED] wrote: Author: sichen Date: Tue Apr 8 16:21:14 2008 New Revision: 646132 URL: http://svn.apache.org/viewvc?rev=646132view=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=646132r1=646131r2=646132view=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 descriptionCreate 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
+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, [EMAIL PROTECTED] wrote: Author: sichen Date: Tue Apr 8 16:21:14 2008 New Revision: 646132 URL: http://svn.apache.org/viewvc?rev=646132view=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=646132r1=646131r2=646132view=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 descriptionCreate 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
+1 Scott On 13/04/2008, Jacopo Cappellato [EMAIL PROTECTED] 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, [EMAIL PROTECTED] wrote: Author: sichen Date: Tue Apr 8 16:21:14 2008 New Revision: 646132 URL: http://svn.apache.org/viewvc?rev=646132view=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=646132r1=646131r2=646132view=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 descriptionCreate 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
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, [EMAIL PROTECTED] wrote: Author: sichen Date: Tue Apr 8 16:21:14 2008 New Revision: 646132 URL: http://svn.apache.org/viewvc?rev=646132view=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=646132r1=646131r2=646132view=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 descriptionCreate 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/