Jacopo Cappellato wrote:
Si Chen wrote:
Jacopo,
Also this script probably may not be handling serialized inventory
correctly. I did not check it yesterday but it needs to be checked.
In general I am concerned that this script means the re-writing the
business logic of the inventory services here in the BSH.
Si
yes, these are two good points too; however I think that sometimes
it's worth having to maintain some duplicated business logics if there
are serious performance issues (as it is happening here).
Jacopo
For long term maintenance and to avoid bugs, we should really try to
keep all the business logic of inventory calculation in one place. The
current services can be made a lot more efficient. In reality, though,
on a real server the performance is not too bad even with them the way
they are.
Si
