ConvertVDB is still adding create foreign data wrapper x; statements. These are not needed and should be suppressed. It's also causing some confusion that a user can enter

create foreign data wrapper x options (...);

The options are actually ignored. We should probably throw an exception instead. Either a type or a handler are required (we don't currently support a separate notion of a handler - it's treated the same as type).

See also TEIID-5850