TEIIDDES-1660 added check in createKey() for columns that are Nullable or undefined, and change to Not Nullable
TEIIDDES-1699 Reworded Dialog message to clarify no Teiid Servers
Merge pull request #150 from mdrillin/master

TEIIDDES-1699
TEIIDDES-1670 removed the "Set Description..." checkbox because difference logic for descriptions is wrong.
Need to assess scope of potential fix and when to fix
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.
TEIIDDES-1717 Allows alternate file extensions in Flat File Importer
Merge pull request #152 from mdrillin/TEIIDDES-1717

TEIIDDES-1717 Allows alternate file extensions in Flat File Importer
Fixes the possibility of a teiid version having additional components

* Code naively assumes that a teiid server version will have a maximum of
  3 components. However, version such as 8.4.0.beta2 has 4 components
  causing the minor and micro version fields to not be set, resulting in
  incorrect assignment of runtimes clients.

* This addresses this possibility by ignoring remaining components but
  ensuring that the first 3 components have been correctly set.
TEIIDDES-1696 points runtime client to upgrade to 8.2.0 SNAPSHOT
TEIIDDES-1710: Adds teiid 8.4 runtime client submodule and test plugin

* Creates the submodule pointing to the 8.4.x branch of designer's teiid
  runtime clients.

* Adds the 8.4 runtime client test plugin
TEIIDDES-1701: Small bug fixes for refactoring functions

* RelatedResourceFinder
 * Typo for ALL relationship.

* RefactorResourceUtils
 * Caters for real possibility that a group name appears in an SQL statement
   line more than once, eg. SELECT SourceModel.ID, SourceModel.name, rather
   than only finding the first.

* RenameResourceRefactoring
 * Only if SQL edits have been added to the TextFileChange should it be
   added as a change. Otherwise, an empty change object is added and makes
   no sense.