[FLINK-9570] SQL Client merging environments uses AbstractMap

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.5.0
• Fix Version/s: 1.5.1, 1.6.0
• Assignee: Dominik Wosiński
• Reporter: Dominik Wosiński
• Created: 12/Jun/18 08:42
• Updated: 13/Jun/18 13:22
• Resolved: 13/Jun/18 13:22

Description

Currently Environment.merge() function looks like below: 

 

final Environment mergedEnv = new Environment();

// merge tables
final Map<String, TableDescriptor> tables = new HashMap<>(env1.getTables());
mergedEnv.getTables().putAll(env2.getTables());
mergedEnv.tables = tables;

and no-arg constructor for Environment defaults tables to Collections.emptyMap().
This basically results in calling putAll on EmptyMap which defaults to AbstractMap which always throws UnsuppoertedOperationException.

 

Issue Links

• links to: GitHub Pull Request #6157