[MDSAL-538] Improve DataObjectStreamer/cache serialization Created: 21/Apr/20 Updated: 16/Jun/21 |
|
| Status: | Open |
| Project: | mdsal |
| Component/s: | Binding runtime |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Medium |
| Reporter: | Robert Varga | Assignee: | Unassigned |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | pt | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
DataObjectStreamer.commonStreamList() can end up performing a lot of instanceof checks to check if the writer can perform caching. For large lists this is wasteful – we should be able to just check half of the condition just once. Note that the logic here is very shallow, so JIT may actually be optimizing this case very well by hoisting it. Add a benchmark to see if our investment in complicating the code by hoisting the check manually actually makes sense. |