[
https://issues.apache.org/jira/browse/OFBIZ-11796?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17127952#comment-17127952
]
Jacques Le Roux commented on OFBIZ-11796:
-----------------------------------------
Hi Suraj,
Weird that nobody thought about that before. I guess nobody crossed performance
issue before, excellent spot Suraj: +1 for commit (only reviewed, but so simple
I'm confident)
> Unnecessary iterations for all productFacilities in setLastInventoryCount
> -------------------------------------------------------------------------
>
> Key: OFBIZ-11796
> URL: https://issues.apache.org/jira/browse/OFBIZ-11796
> Project: OFBiz
> Issue Type: Bug
> Components: product
> Affects Versions: Release Branch 18.12, Release Branch 17.12, Trunk
> Reporter: Suraj Khurana
> Assignee: Suraj Khurana
> Priority: Major
> Attachments: OFBIZ-11796.patch
>
>
> In the service setLastInventoryCount, which updates lastInventoryCount for
> product facility records, it works on inventoryItemDetail changes on
> availableToPromiseDiff.
> In parameter: InventoryItemId
> So it should be updating productFacilities entertaining both productId and
> facilityId of same inventoryItemId.
> Currently, it it unnecessary updating all the facilities inventory on a
> single inventory item update.
> It can be major performance factor on a production system working highly on
> inventory updates.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
