Reuse SegmentEntryIds from the HTTPRequest

• Type: Bug
• Status: Closed
• Resolution: Fixed
• Affects Version/s: Master
• Fix Version/s: 7.2.X, 7.3.10 DXP FP1, 7.3.10.1 DXP SP1, 7.3.6 CE GA7, 7.3.X, Master
• Component/s: Segments Editor
• Labels:
  • ee-ts
  • liferay-dxp-73-sp1
  • liferay-fixpack-dxp-1-7310
  • liferay-fixpack-dxp-9-7210
• Branch Version/s:
  7.3.x, 7.2.x
• Backported to Branch:
  Committed
• Fix Priority: 3
• Epic Link: tango-segmentation-performance
• Git Pull Request: https://github.com/shuyangzhou/liferay-portal/pull/9523 https://github.com/liferay-tango/liferay-portal/pull/563

Description

We have executed the SignIn performance test with 100 segments, and the result shows a very important regression, you can check the full result in the table below:

Portal Version	Git Id	Session Count	Mean Time of Login	Portal CPU Usage (%)	DB CPU Usage (%)	Session count changes(%)
BaseLine	2b20ab5	38428	269	96,96,96	0,0,0
Segment(100 segments)	2b20ab5	2453	572	56,56,56	0,0,0	-93.6166337

You can find the full results here

Also, you can download the profile from here.

Proposed solution

Reuse the parameter SegmentEntryIds from the HttpRequest in ServicePreaction and SegmentsRoleContributor