[teiid-designer-dev] [Teiid-Designer/teiid-designer] f6e039: TEIIDDES-2299: Improve the performance of ldap wiz...

[Paul Richardson]

  Branch: refs/heads/8.6.x
  Home:   https://github.com/Teiid-Designer/teiid-designer
  Commit: f6e039bb4596d27c9e94a4bd9978b2d0e7da18f2
      https://github.com/Teiid-Designer/teiid-designer/commit/f6e039bb4596d27c9e94a4bd9978b2d0e7da18f2
  Author: Paul Richardson <p.g.richardson at redhat.com>
  Date:   2015-01-27 (Tue, 27 Jan 2015)

  Changed paths:
    M plugins/org.teiid.designer.modelgenerator.ldap.ui/src/org/teiid/designer/modelgenerator/ldap/ui/wizards/LdapImportWizardManager.java
    M plugins/org.teiid.designer.modelgenerator.ldap.ui/src/org/teiid/designer/modelgenerator/ldap/ui/wizards/pages/columns/LdapColumnsPage.java
    M plugins/org.teiid.designer.modelgenerator.ldap.ui/src/org/teiid/designer/modelgenerator/ldap/ui/wizards/pages/columns/LdapEntryLabelProvider.java
    M plugins/org.teiid.designer.modelgenerator.ldap.ui/src/org/teiid/designer/modelgenerator/ldap/ui/wizards/pages/table/LdapConnectionLabelProvider.java
    M plugins/org.teiid.designer.modelgenerator.ldap.ui/src/org/teiid/designer/modelgenerator/ldap/ui/wizards/pages/table/LdapTablesPage.java

  Log Message:
  -----------
  TEIIDDES-2299: Improve the performance of ldap wizard checkbox selection

* LdapImportWizardManager
 * Stop setSynchronising from notifying that it has changed as this is
   calling update in the middle of user-selection operations. Not least
   it is simply a flag that is observed via isSynchronising()

* Ldap[Columns|Tables]Page
 * Refactor the tree viewers of these pages to be CheckboxTreeViewers as
   this avoids the expensive method of WidgetUtil.findTreeItem().
 * With the change to setSynchronising the performance of the trees is much
   improved