[jboss-jira] [JBoss JIRA] (DROOLS-4735) [DMN Designer] Grid performance is dire

[Michael Anstis (Jira)]

    [ https://issues.jboss.org/browse/DROOLS-4735?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13814863#comment-13814863 ] 

Michael Anstis commented on DROOLS-4735:
----------------------------------------

[~jomarko] Your last comment about editing a cell on a scrolled table may be fixed by https://github.com/kiegroup/appformer/pull/848

That PR is "on top of" the PR for this JIRA/grid performance so you might be able to check/test #848 in conjunction with this..

> [DMN Designer] Grid performance is dire
> ---------------------------------------
>
>                 Key: DROOLS-4735
>                 URL: https://issues.jboss.org/browse/DROOLS-4735
>             Project: Drools
>          Issue Type: Bug
>          Components: DMN Editor
>    Affects Versions: 7.29.0.Final
>            Reporter: Michael Anstis
>            Assignee: Michael Anstis
>            Priority: Critical
>              Labels: drools-tools
>         Attachments: aTable.dmn, dtable-500-rows.dmn, edit-cell.webm
>
>
> There have been reports that the performance of the DMN editor's grids does not scale. On examination performance for anything other than simple examples is pretty dire.. something is seriously wrong.
> On closer inspection the following issues were found:
> - Multiple {{NodeMouseDownHandler}}, {{NodeMouseMoveHandler}} and {{NodeMouseUpHandler}} are registered because multiple {{DMNGridLayer}} are (incorrectly) instantiated due to a flaw in {{GridLienzoPanel}} and {{GridLienzoScrollHandler}}.
> - Calls to {{GridWidget.getHeight()}} and {{GridRow.getHeight()}} can be _very_ expensive for due to the dynamic calculation in {{ExpressionEditorGridRow}} and {{LiteralExpressionGridRow}} rows.
> h2. Manual Acceptance test
> Environment: wildfly14 with following memory settings: {{JAVA_OPTS="-Xms64m -Xmx1024m -XX:MetaspaceSize=96M -XX:MaxMetaspaceSize=1024m -Djava.net.preferIPv4Stack=true"}}
> h3. GDT
> - Merge view (/)
> - Pinned mode (/)
> - Change column width, save and reopen (/)
> - Search the table (/)
> - Scroll the table (/)
> - Edit the cell (/)
> - Reorder columns (/)
> - Reorder rows (/)
> h3. DMN decision table
> - Save and reopen (x) not able to save in reasonable time
> - Change column width (/)
> - Search the grid (/)  
> - Scroll the grid (/)
> - Reorder columns (/)
> - Reorder rows (/)
> - Adding column (/)
> - Removing column (/) 
> - Edit the cell (x) not able to invoke edit mode by double click, if the table was scrolled   [^edit-cell.webm] 



--
This message was sent by Atlassian Jira
(v7.13.8#713008)