Ryan Fu created CALCITE-3436: -------------------------------- Summary: CalciteConnectionConfigImpl.set obliterates previous property values Key: CALCITE-3436 URL: https://issues.apache.org/jira/browse/CALCITE-3436 Project: Calcite Issue Type: Bug Reporter: Ryan Fu
The cause is that `new Properties(properties)` doesn't copy the previous value as, say, `new HashMap(map)` does. * Add a unit test in PlannerTest, that if you call set(x, 1) then set(y, 2) on a CalciteConnectionConfigImpl, the result contains x and y. * Consider potentially using `clone()` as Properties implements Cloneable -- This message was sent by Atlassian Jira (v8.3.4#803005)