Re: Entity Caching

2015-03-18 Thread Scott Gray
I tend to disagree with the "cache everything" approach because the cache isn't transaction aware. If you: 1. update a record 2. select that same record 3. encounter a transaction rollback Then the cache will still contain the changes that were rolled back. Regards Scott On Wed, Mar 18, 2015 at

[jira] [Updated] (OFBIZ-6175) Replace findInventoryEventPlan.ftl with form widgets

2015-03-18 Thread Christian Carlow (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-6175?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Christian Carlow updated OFBIZ-6175: Attachment: OFBIZ-6175.patch > Replace findInventoryEventPlan.ftl with form widgets > --

[jira] [Updated] (OFBIZ-6175) Replace findInventoryEventPlan.ftl with form widgets

2015-03-18 Thread Christian Carlow (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-6175?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Christian Carlow updated OFBIZ-6175: Attachment: (was: OFBIZ-6175.patch) > Replace findInventoryEventPlan.ftl with form widge

[jira] [Updated] (OFBIZ-6175) Replace findInventoryEventPlan.ftl with form widgets

2015-03-18 Thread Christian Carlow (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-6175?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Christian Carlow updated OFBIZ-6175: Attachment: OFBIZ-6175.patch This patch replaces the need for findInventoryEventPlan.ftl and

[jira] [Created] (OFBIZ-6175) Replace findInventoryEventPlan.ftl with form widgets

2015-03-18 Thread Christian Carlow (JIRA)
Christian Carlow created OFBIZ-6175: --- Summary: Replace findInventoryEventPlan.ftl with form widgets Key: OFBIZ-6175 URL: https://issues.apache.org/jira/browse/OFBIZ-6175 Project: OFBiz Issu

[jira] [Updated] (OFBIZ-5949) ofbizsetup create product store cause exception, remove use of temp file to load entity data

2015-03-18 Thread Wai (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Wai updated OFBIZ-5949: --- Attachment: OFBIZ-5949.patch > ofbizsetup create product store cause exception, remove use of temp file to > load ent

[jira] [Commented] (OFBIZ-6174) Show all production runs associated with order items

2015-03-18 Thread Christian Carlow (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-6174?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14367862#comment-14367862 ] Christian Carlow commented on OFBIZ-6174: - Maybe a better solution would be to sho

[jira] [Commented] (OFBIZ-6174) Show all production runs associated with order items

2015-03-18 Thread Christian Carlow (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-6174?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14367810#comment-14367810 ] Christian Carlow commented on OFBIZ-6174: - Hey Pierre, I suppose it is premature,

[jira] [Commented] (OFBIZ-5949) ofbizsetup create product store cause exception, remove use of temp file to load entity data

2015-03-18 Thread Jacques Le Roux (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14367767#comment-14367767 ] Jacques Le Roux commented on OFBIZ-5949: Yes, the problem with "philosophy" is you

[jira] [Comment Edited] (OFBIZ-5949) ofbizsetup create product store cause exception, remove use of temp file to load entity data

2015-03-18 Thread Pierre Smits (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14367739#comment-14367739 ] Pierre Smits edited comment on OFBIZ-5949 at 3/18/15 7:34 PM: --

[jira] [Commented] (OFBIZ-5949) ofbizsetup create product store cause exception, remove use of temp file to load entity data

2015-03-18 Thread Pierre Smits (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14367739#comment-14367739 ] Pierre Smits commented on OFBIZ-5949: - Luckily we version control management, so we ca

[jira] [Commented] (OFBIZ-6174) Show all production runs associated with order items

2015-03-18 Thread Pierre Smits (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-6174?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14367733#comment-14367733 ] Pierre Smits commented on OFBIZ-6174: - Hi Christian, If you don't understand the view

[jira] [Commented] (OFBIZ-5949) ofbizsetup create product store cause exception, remove use of temp file to load entity data

2015-03-18 Thread Wai (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-5949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14367706#comment-14367706 ] Wai commented on OFBIZ-5949: The importTempDataFile method was commented out and labeled 'obso

[jira] [Updated] (OFBIZ-6174) Show all production runs associated with order items

2015-03-18 Thread Christian Carlow (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-6174?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Christian Carlow updated OFBIZ-6174: Attachment: OFBIZ-6174.patch > Show all production runs associated with order items > --

[jira] [Updated] (OFBIZ-6174) Show all production runs associated with order items

2015-03-18 Thread Christian Carlow (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-6174?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Christian Carlow updated OFBIZ-6174: Affects Version/s: Trunk > Show all production runs associated with order items > --

[jira] [Created] (OFBIZ-6174) Show all production runs associated with order items

2015-03-18 Thread Christian Carlow (JIRA)
Christian Carlow created OFBIZ-6174: --- Summary: Show all production runs associated with order items Key: OFBIZ-6174 URL: https://issues.apache.org/jira/browse/OFBIZ-6174 Project: OFBiz Issu

[jira] [Closed] (OFBIZ-5984) Extend WorkOrderItemFulfillment to support relations to more than one OrderItemShipGroupAssoc

2015-03-18 Thread Christian Carlow (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-5984?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Christian Carlow closed OFBIZ-5984. --- Resolution: Invalid Closing this issue because production runs per orderItemShipGroupAssoc is

[jira] [Assigned] (OFBIZ-6173) Allow order item quantities to be assigned to ship groups added to existing orders

2015-03-18 Thread Jacques Le Roux (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-6173?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jacques Le Roux reassigned OFBIZ-6173: -- Assignee: Jacques Le Roux > Allow order item quantities to be assigned to ship groups a

Re: Entity Caching

2015-03-18 Thread Ron Wheeler
+1 People often forget that cache memory is hundreds or thousands time faster that disk access or database references. Even with the overhead of distributed cache management. Any extra code required executes at lightspeed rather than rotational speed or even SSD speed. It is also easier to sc

[jira] [Updated] (OFBIZ-6173) Allow order item quantities to be assigned to ship groups added to existing orders

2015-03-18 Thread Christian Carlow (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-6173?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Christian Carlow updated OFBIZ-6173: Attachment: OFBIZ-6173.patch > Allow order item quantities to be assigned to ship groups add

[jira] [Updated] (OFBIZ-6173) Allow order item quantities to be assigned to ship groups added to existing orders

2015-03-18 Thread Christian Carlow (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-6173?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Christian Carlow updated OFBIZ-6173: Issue Type: Improvement (was: Bug) > Allow order item quantities to be assigned to ship gro

[jira] [Created] (OFBIZ-6173) Allow order item quantities to be assigned to ship groups added to existing orders

2015-03-18 Thread Christian Carlow (JIRA)
Christian Carlow created OFBIZ-6173: --- Summary: Allow order item quantities to be assigned to ship groups added to existing orders Key: OFBIZ-6173 URL: https://issues.apache.org/jira/browse/OFBIZ-6173

Re: Entity Caching

2015-03-18 Thread Jacopo Cappellato
+1 On Mar 18, 2015, at 5:16 PM, Jacques Le Roux wrote: > If a such effort would be undertaken, I'd prefer to go with ehcache: > http://ehcache.org/documentation/2.6/get-started/about-distributed-cache > > Jacques > > Le 18/03/2015 14:27, Adrian Crum a écrit : >> I agree there is a lot of ove

Re: Entity Caching

2015-03-18 Thread Jacques Le Roux
If a such effort would be undertaken, I'd prefer to go with ehcache: http://ehcache.org/documentation/2.6/get-started/about-distributed-cache Jacques Le 18/03/2015 14:27, Adrian Crum a écrit : I agree there is a lot of overhead with the current distributed cache clearing implementation. From my

Re: Entity Caching

2015-03-18 Thread Adrian Crum
On 3/18/2015 1:46 PM, Jacopo Cappellato wrote: On Mar 18, 2015, at 2:35 PM, Adrian Crum wrote: I would be interested in seeing a use case for that. This is just an example I could think of (there may be other better examples but I am in a rush and I can't concentrate): Client code A selec

Re: Entity Caching

2015-03-18 Thread Jacopo Cappellato
On Mar 18, 2015, at 2:35 PM, Adrian Crum wrote: > I would be interested in seeing a use case for that. This is just an example I could think of (there may be other better examples but I am in a rush and I can't concentrate): Client code A selects a bunch of records from TableFoo in order to u

Re: Entity Caching

2015-03-18 Thread Adrian Crum
I would be interested in seeing a use case for that. I view delegator calls as being no different than SQL statements. Where in the SQL grammar do you tell the database to get a table row from its cache or not? That detail is left to the implementation, and in some cases it can be controlled v

Re: Entity Caching

2015-03-18 Thread Adrian Crum
I agree there is a lot of overhead with the current distributed cache clearing implementation. From my perspective, that should be changed to use RMI - where the caches talk directly to each other. Adrian Crum Sandglass Software www.sandglass-software.com On 3/18/2015 1:21 PM, Rupert Howell wr

Re: Entity Caching

2015-03-18 Thread Jacopo Cappellato
On Mar 18, 2015, at 1:16 PM, Adrian Crum wrote: > If you code Delegator calls to avoid the cache, then there is no way for a > sysadmin to configure the caching behavior - that bit of code will ALWAYS > make a database call. > > If you make all Delegator calls use the cache, then there is an

Re: Entity Caching

2015-03-18 Thread Rupert Howell
Yes I'm aware of this. If everything was cached Surely every persist would need to trigger a JMS message telling all the other servers to dump their local cache. This overhead would be big, the amount of messages flying out would be hugely increased and the caches would be redundant anyway because

Re: Entity Caching

2015-03-18 Thread Adrian Crum
A clustered environment must use distributed cache clearing. Adrian Crum Sandglass Software www.sandglass-software.com On 3/18/2015 12:34 PM, Rupert Howell wrote: Hi Adrian, How would this work in a clustered or load balanced environment? With multiple nodes always checking their own local cac

Re: Entity Caching

2015-03-18 Thread Rupert Howell
Hi Adrian, How would this work in a clustered or load balanced environment? With multiple nodes always checking their own local caches incorrect data would be persisted all the time. Regards, Rupert On 18 March 2015 at 12:16, Adrian Crum wrote: > I would like to share some insights into the e

Entity Caching

2015-03-18 Thread Adrian Crum
I would like to share some insights into the entity cache feature, some best practices I like to follow, and some related information. Some OFBiz experts may disagree with some of my views, and that is okay. Different experiences with OFBiz will lead to different viewpoints. The OFBiz entity

Re: svn commit: r1667495 - /ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderReadHelper.java

2015-03-18 Thread Deepak Dixit
Thanks Adrian for code review, I found warning on console, and found that in all places “OrderItemBilling” record was fetched with cache=false, if we want to override the OOTB behavior then we have to override the fetching logic as well. I found interesting thread: http://markmail.org/message/oqf

Re: svn commit: r1667495 - /ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderReadHelper.java

2015-03-18 Thread Pierre Smits
If contributors do not understand such features, maybe some additional explanation in our wiki (or hints thereto) are in order. Best regards, Pierre Smits *ORRTIZ.COM * Services & Solutions for Cloud- Based Manufacturing, Professional Services and Retail & Trade http://www

Re: svn commit: r1667483 - in /ofbiz/trunk: applications/order/entitydef/entitymodel.xml specialpurpose/scrum/entitydef/entitymodel.xml

2015-03-18 Thread Pierre Smits
Hi Deepak, That issue shortly after the moment release r13.07.01 came available to the public. That release didn't have numerous special purpose components incorporated. And made that bug visible. However, though something is uses at a lower level, we should not have something as basic as a paren

Re: svn commit: r1667483 - in /ofbiz/trunk: applications/order/entitydef/entitymodel.xml specialpurpose/scrum/entitydef/entitymodel.xml

2015-03-18 Thread Jacques Le Roux
Hi Deepak, I'm not sure if we should or not revisit https://issues.apache.org/jira/browse/OFBIZ-5481 and backport more (just spotted it, did not re-read comments there yet) Jacques Le 18/03/2015 11:33, dee...@apache.org a écrit : Author: deepak Date: Wed Mar 18 10:33:39 2015 New Revision: 16

Re: svn commit: r1667495 - /ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderReadHelper.java

2015-03-18 Thread Adrian Crum
No. The attribute is in the entity definition, so it is a global setting. The problem here is contributors do not understand the entity caching feature, and we end up with conflicting settings, or settings that don't make any sense. Adrian Crum Sandglass Software www.sandglass-software.com O

Re: svn commit: r1667495 - /ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderReadHelper.java

2015-03-18 Thread Pierre Smits
Should this not be configurable per tenant? Pierre Smits *ORRTIZ.COM * Services & Solutions for Cloud- Based Manufacturing, Professional Services and Retail & Trade http://www.orrtiz.com On Wed, Mar 18, 2015 at 12:04 PM, Adrian Crum < adrian.c...@sandglass-software.com> wr

[jira] [Closed] (OFBIZ-6172) Screenlet collapsibility is not working

2015-03-18 Thread Adrian Crum (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-6172?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Adrian Crum closed OFBIZ-6172. -- Resolution: Fixed Fix Version/s: Upcoming Branch 14.12.01 Fixed in revs 166749

Re: svn commit: r1667483 - in /ofbiz/trunk: applications/order/entitydef/entitymodel.xml specialpurpose/scrum/entitydef/entitymodel.xml

2015-03-18 Thread Deepak Dixit
Hi Pierre, As parentCustRrequestId is only used in Scrum component so it does not make sense to add this in main entity definitions https://issues.apache.org/jira/browse/OFBIZ-5481, Thanks & Regards — Deepak Dixit > On Mar 18, 2015, at 4:19 PM, Pierre Smits wrote: > > Though the 'parentCus

Re: svn commit: r1667495 - /ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderReadHelper.java

2015-03-18 Thread Adrian Crum
What about when I decide to set never-cache to false on my local copy? Adrian Crum Sandglass Software www.sandglass-software.com On 3/18/2015 11:01 AM, dee...@apache.org wrote: Author: deepak Date: Wed Mar 18 11:01:25 2015 New Revision: 1667495 URL: http://svn.apache.org/r1667495 Log: Fixed Or

Re: svn commit: r1667483 - in /ofbiz/trunk: applications/order/entitydef/entitymodel.xml specialpurpose/scrum/entitydef/entitymodel.xml

2015-03-18 Thread Pierre Smits
Though the 'parentCustRequestId' is used particularly in the SCRUM component, the CustRequest entity is defined in ORDERMGR and extended in SCRUM. I would say that moving the definition of the field 'parentCustRequestId' from SCRUM to ORDERMGR would be the better solution, given that there are sim

[jira] [Updated] (OFBIZ-6172) Screenlet collapsibility is not working

2015-03-18 Thread Thawatchai Klinkliang (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-6172?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Thawatchai Klinkliang updated OFBIZ-6172: - Affects Version/s: Trunk > Screenlet collapsibility is not working > -

[jira] [Assigned] (OFBIZ-6172) Screenlet collapsibility is not working

2015-03-18 Thread Adrian Crum (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-6172?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Adrian Crum reassigned OFBIZ-6172: -- Assignee: Adrian Crum > Screenlet collapsibility is not working > -

[jira] [Issue Comment Deleted] (OFBIZ-6172) Screenlet collapsibility is not working

2015-03-18 Thread Thawatchai Klinkliang (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-6172?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Thawatchai Klinkliang updated OFBIZ-6172: - Comment: was deleted (was: issue fixed patch) > Screenlet collapsibility is not w

[jira] [Updated] (OFBIZ-6172) Screenlet collapsibility is not working

2015-03-18 Thread Thawatchai Klinkliang (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-6172?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Thawatchai Klinkliang updated OFBIZ-6172: - Attachment: OFBIZ-6172_ModelScreenWidget.patch issue fixed patch > Screenlet coll

[jira] [Created] (OFBIZ-6172) Screenlet collapsibility is not working

2015-03-18 Thread Thawatchai Klinkliang (JIRA)
Thawatchai Klinkliang created OFBIZ-6172: Summary: Screenlet collapsibility is not working Key: OFBIZ-6172 URL: https://issues.apache.org/jira/browse/OFBIZ-6172 Project: OFBiz Issue T

[jira] [Updated] (OFBIZ-6171) Replace 'Open for Business' references with 'Apache OFBiz'

2015-03-18 Thread Pierre Smits (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-6171?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Pierre Smits updated OFBIZ-6171: Description: Currently the project is referenced in various files with 'Open For Business'. This sho

[jira] [Created] (OFBIZ-6171) Replace 'Open for Business' references with 'Apache OFBiz'

2015-03-18 Thread Pierre Smits (JIRA)
Pierre Smits created OFBIZ-6171: --- Summary: Replace 'Open for Business' references with 'Apache OFBiz' Key: OFBIZ-6171 URL: https://issues.apache.org/jira/browse/OFBIZ-6171 Project: OFBiz Issue

[jira] [Commented] (OFBIZ-6170) You can't rely on Lookup field description-field-name attribute in a form having skip-start="true"

2015-03-18 Thread Jacques Le Roux (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-6170?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14366809#comment-14366809 ] Jacques Le Roux commented on OFBIZ-6170: Initially I worked on a Java solution, bu

[jira] [Comment Edited] (OFBIZ-6170) You can't rely on Lookup field description-field-name attribute in a form having skip-start="true"

2015-03-18 Thread Jacques Le Roux (JIRA)
[ https://issues.apache.org/jira/browse/OFBIZ-6170?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14366809#comment-14366809 ] Jacques Le Roux edited comment on OFBIZ-6170 at 3/18/15 8:27 AM: ---