The cache for IDisplayTypeFactory created with ticket https://idempiere.atlassian.net/browse/IDEMPIERE-4498 shows too many misses which can be bad for performance.
For example at this moment checking here:
https://test.idempiere.org/idempiereMonitor?CacheDetails=Yes
shows 333.461 misses and 0 hits
https://demo.globalqss.com/idempiereMonitor?CacheDetails=Yes
shows 12.640.292 misses and 0 hits.
A better approach could be caching the miss as a null.
Also, reviewing the DisplayType class found some inconsistencies in the return of the different functions using the factory.
The cache for IDisplayTypeFactory created with ticket https://idempiere.atlassian.net/browse/IDEMPIERE-4498 shows too many misses which can be bad for performance.
For example at this moment checking here:
https://test.idempiere.org/idempiereMonitor?CacheDetails=Yes
shows 333.461 misses and 0 hits
https://demo.globalqss.com/idempiereMonitor?CacheDetails=Yes
shows 12.640.292 misses and 0 hits.
A better approach could be caching the miss as a null.
Also, reviewing the DisplayType class found some inconsistencies in the return of the different functions using the factory.