TEIIDDES-1677 Improved VDB Editor layout for models tab
Merge pull request #151 from mdrillin/TEIIDDES-1677

TEIIDDES-1677
TEIIDDES-1709 moved addChild() call before all setProperties() calls while walking through model object creation
TEIIDDES-1701: Tighten up the checks on resources before refactoring

* The error-checking on resources and their dependents prior to their
  refactoring is inconsistent and error-prone.

* UiUtil
 * Check all workbench window and pages for any editors using the given
   file

* AbstractResourcesRefactoring
 * Removes workbench window field as unnecessary
 * Migrates all check methods to RefactorResourceUtils
 * Introduces abstract checkResource method that should implement all
   checks required on a given resource for the specific refactoring use-case
 * Removes the closeDirtyEditors method due to modification the saving
   function creates separate runnables in their own threads leading to
   problems with synchronisation. Instead, if their are open editors or
   dirty resources, the refactoring bails with a message to the user

* RefactorResourcesUtils
 * Renames the resource callback interface to IResourceCallback and removes
   its mergeStatus method since it should not hold a status object of its
   own
 * Provides abstract implementation of IResourceCallback
 * Introduces check for open editors

* VdbModelEntry
 * Avoid NPE by checking if we have derived a model resource
TEIIDDES-1703: Make it obvious what the delete action will cover

* Places a tree viewer on the 'Are you sure?' dialog box displaying all the
  resources and dependencies that will be deleted by the delete refactoring

* Makes it plain to the user that these resources will be deleted if they
  click ok but can deselect them if they click preview.
TEIIDDES-1712: Refactors and simplifies object delete command

* Remove the remaining complexity of the ObjectDeleteCommand by merging it
  with its parent class ResourceRefactorCommand. Since the latter is only
  used now by the former, its unnecessary. The parent class is geared
  towards resources rather than eObjects so it makes little sense for the
  ObjectDeleteCommand to extend it anyway.

* ResourceRefactorCommand deleted and references removed
TEIIDDEs-1701: Incorrect relationship used during rename and move

* Move and Rename Resource Refactorings are using the wrong relationship
  for finding other resources to modify. Like delete, they should be
  finding their dependents and not their dependencies.
TEIIDDES-1718 removed import for wrong RelationalObjectEditorFactory
Updates to latest changes in teiid 8.3.x

* Limits the teiid server versions that 8.3 client supports

* Removes unnecessary dependencies from pom
Fix DisplayNodeFactory tests in light of 8.3 teiid runtime version change

* By limiting the 8.3 teiid runtime to specific versions, ie. 8.0.x, 8.1.x,
  8.2.x and 8.3.x, the tests fail since the tests cannot find expected
  results for anything other than 8.2.0.

* Extends the tests to specifically cover the other expected teiid runtime
  version.