Bug 1856355 - Scrolling of pf4 tables is far less performant than the previous version
Summary: Scrolling of pf4 tables is far less performant than the previous version
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Management Console
Version: 4.5
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ---
: 4.9.0
Assignee: Rastislav Wagner
QA Contact: Yadan Pei
URL:
Whiteboard: Scrubbed
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-07-13 13:01 UTC by Paul Weil
Modified: 2021-10-18 17:29 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Cause: Virtualized table passes new style prop on every scroll event which casues all visible rows to rerender Consequence: bad performance sice too many elements have to rerender Fix: table row content was wrapped in React.memo Result: table row content does not rerender on scroll event
Clone Of:
Environment:
Last Closed: 2021-10-18 17:28:52 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github openshift console pull 9797 0 None None None 2021-08-12 12:56:42 UTC
Red Hat Product Errata RHSA-2021:3759 0 None None None 2021-10-18 17:29:06 UTC

Description Paul Weil 2020-07-13 13:01:30 UTC
In comparing the new PF4 tables to the previous version, I noticed scrolling is noticeably degraded. The PF4 table scrolling is choppy and jumpy by comparison.

See https://issues.redhat.com/browse/CONSOLE-1532 for screenshot

Comment 1 Joe Caiani 2020-07-31 20:28:59 UTC
A bug to fix the table filter has been fixed and the issue that relates to this has also been closed. adding a separate issue for table latency in the pf repo.

Comment 5 Jakub Hadvig 2020-12-23 16:10:25 UTC
We did not have time to fix this issue this sprint. Will reevaluate and try to fix in next sprint.

Comment 10 Samuel Padgett 2021-04-30 17:49:20 UTC
We are working on a new table in https://github.com/openshift/console/pull/8829#issuecomment-830248046 and might be able to look at improving scrolling performance there.

Comment 12 Samuel Padgett 2021-07-23 19:22:27 UTC
We are blocked until this is fixed in PatternFly.

Comment 14 Yadan Pei 2021-08-23 08:04:25 UTC
Goes to pods list page with 'All Projects' selected /k8s/all-namespaces/pods, scrolling up and down, notice the performance, no big performance issue here.

Verified on 4.9.0-0.nightly-2021-08-22-070405

Let me know if there is more functions need to be checked

Comment 17 errata-xmlrpc 2021-10-18 17:28:52 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Moderate: OpenShift Container Platform 4.9.0 bug fix and security update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2021:3759


Note You need to log in before you can comment on or make changes to this bug.